Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add description and examples for $id keyword (#105) #106

Merged
merged 3 commits into from
Feb 6, 2024

Conversation

AgniveshChaubey
Copy link
Collaborator

@AgniveshChaubey AgniveshChaubey commented Feb 6, 2024

This PR addresses #47.

It is built upon the changes requested in #45 and implements the suggested modifications

Also let me know what you think about this card.
image

This PR addresses #47.

It is built upon the changes proposed in #45 and implements the
suggested modifications

---------

Co-authored-by: Juan Cruz Viotti <[email protected]>
Copy link
Member

@jviotti jviotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool! I left some very minor suggestions by approving already

content/2020-12/core/id.markdown Outdated Show resolved Hide resolved
content/2020-12/core/id.markdown Outdated Show resolved Hide resolved
@jviotti
Copy link
Member

jviotti commented Feb 6, 2024

BTW, if you are up for it, you should consider upstreaming the definitions of schemas, schema resources, and schema objects to the JSON Schema official glossary cc @Julian

@jviotti
Copy link
Member

jviotti commented Feb 6, 2024

Later, if they are the glossary, we can probably explain them less here and link there instead?

@AgniveshChaubey
Copy link
Collaborator Author

BTW, if you are up for it, you should consider upstreaming the definitions of schemas, schema resources, and schema objects to the JSON Schema official glossary cc @Julian

Sure, I'm happy to help with that. Thanks for the suggestion!

@AgniveshChaubey AgniveshChaubey merged commit c50ec56 into main Feb 6, 2024
1 check passed
@AgniveshChaubey AgniveshChaubey deleted the keyword-id branch February 6, 2024 16:39
@AgniveshChaubey AgniveshChaubey mentioned this pull request Feb 6, 2024
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