Skip to content

add link to Use Cases doc, remove use cases section. #1429

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

Merged
merged 2 commits into from
Feb 10, 2024

Conversation

brentzundel
Copy link
Member

@brentzundel brentzundel commented Feb 2, 2024

This PR fixes #1169 by removing the use cases section from the data model spec and adding a link to the use cases and requirements doc.


Preview | Diff

Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

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

+1 to @TallTed's changes.

Goodbye list of "things we wanted to accomplish with this architecture", you served us well over the past 7+ years.

@msporny msporny added the editorial Purely editorial changes to the specification. label Feb 4, 2024
@TallTed
Copy link
Member

TallTed commented Feb 5, 2024

Applying my remaining change request will also trigger PR-Preview, which should now work as tobie/pr-preview#149 has been fixed.

Co-authored-by: Ted Thibodeau Jr <[email protected]>
@msporny msporny merged commit 267dee5 into w3c:main Feb 10, 2024
@msporny
Copy link
Member

msporny commented Feb 10, 2024

Editorial, multiple positive reviews, no changes requested, no objections, merging.

@msporny msporny added the CR1 This item was processed during CR1 label Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CR1 This item was processed during CR1 editorial Purely editorial changes to the specification.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider removing Use Cases summary from core DM
6 participants