Skip to content
This repository was archived by the owner on Nov 2, 2023. It is now read-only.

Clarify the meaning of the draft qualifier #399

Merged
merged 4 commits into from
Jul 6, 2021

Conversation

aucampia
Copy link
Contributor

@aucampia aucampia commented Jul 4, 2021

This relates to this discussion: json-schema-org/community#25

This is my understanding from discussions and the presentation made by Ben Hutton.

Feedback welcome, I realize that this is something quite critical and should be changed with care and consideration so I expect that there will be significant changes to this before it is merged.

@netlify
Copy link

netlify bot commented Jul 4, 2021

✔️ Deploy Preview for condescending-hopper-c3ed30 ready!

🔨 Explore the source changes: 11d2ab1

🔍 Inspect the deploy log: https://app.netlify.com/sites/condescending-hopper-c3ed30/deploys/60e4885c7f2e14000759a349

😎 Browse the preview: https://deploy-preview-399--condescending-hopper-c3ed30.netlify.app/

@aucampia aucampia force-pushed the iwana-project_status branch from c7107bd to a7ae96b Compare July 4, 2021 19:24
Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

This sounds great to me. Thank you for taking the time to write it up! I just had one small wording suggestion.

Fix grammar

Co-authored-by: Jason Desrosiers <[email protected]>
@aucampia aucampia requested a review from jdesrosiers July 5, 2021 20:20
@gregsdennis
Copy link
Member

I'm quite happy with this. Definitely a clarification that needs to be made. Thanks for writing it up.

Also change "qualifier" to "designation", and fix grammar in third
paragraph.
Copy link
Contributor

@handrews handrews left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Member

@Relequestual Relequestual left a comment

Choose a reason for hiding this comment

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

Single comment change request

add oxford comma

Co-authored-by: Ben Hutton <[email protected]>
@Relequestual Relequestual merged commit af8d9ce into json-schema-org:master Jul 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants