Skip to content

Add docs for new schema generator feature #594

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

Conversation

smyrick
Copy link
Contributor

@smyrick smyrick commented Feb 3, 2020

📝 Description

Update docs with new page describing the features

🔗 Related Issues

Added new features in #593 that we did not document

Added new features in ExpediaGroup#593 that we did not document
@smyrick smyrick added the type: documentation Documentation or test changes label Feb 3, 2020
@@ -1,6 +1,6 @@
---
id: evolving-schema
title: Evolving Schema
id: deprecating-schema
Copy link
Collaborator

Choose a reason for hiding this comment

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

lets keep it as more generic evolving schema and add notes about adding new fields etc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was mainly doing this to help the sidebar be a little more clear. Right now it is not super descriptive for what is there.

Do you think we can just include this all on one page?

Screen Shot 2020-02-03 at 2 48 46 PM

Copy link

Choose a reason for hiding this comment

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

The only info in the topic right now is on deprecating schema elements, so I'd recomment the title: Deprecating Schema Elements.

If or when you talk about other ways to evolve the schema or its elements, I'd change to something like Evolving Schema.

Just my two cents.

@dariuszkuc dariuszkuc merged commit 999c555 into ExpediaGroup:master Feb 7, 2020
@smyrick smyrick deleted the document-new-schema-generator branch February 7, 2020 17:29
dariuszkuc pushed a commit to dariuszkuc/graphql-kotlin that referenced this pull request Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation Documentation or test changes
Development

Successfully merging this pull request may close these issues.

2 participants