Skip to content

Adds new CAIP as superset of CAIP-10#152

Closed
bumblefudge wants to merge 4 commits into
masterfrom
feat/caip10-superset
Closed

Adds new CAIP as superset of CAIP-10#152
bumblefudge wants to merge 4 commits into
masterfrom
feat/caip10-superset

Conversation

@bumblefudge
Copy link
Copy Markdown
Collaborator

No description provided.

@oed
Copy link
Copy Markdown
Collaborator

oed commented Oct 18, 2022

Why can't this be an update to caip10?

@oed
Copy link
Copy Markdown
Collaborator

oed commented Oct 18, 2022

If these changes are backwards compatible we should just update caip10 since it's still in draft stage.

@bumblefudge
Copy link
Copy Markdown
Collaborator Author

If these changes are backwards compatible

I thought Ceramic's implementation failed validation when . or % was present? I'm not sure what legacy compatibility issues it would create for other implementers (WC, Spruce, etc.), but this "supersession" approach was the concensus when we talked about it 2 or 3 editorial meetings ago... let's discuss Thursday at the next one!

@oed
Copy link
Copy Markdown
Collaborator

oed commented Oct 18, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants