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

docs/md: various fixes #2965

Merged
merged 5 commits into from Sep 27, 2022
Merged

docs/md: various fixes #2965

merged 5 commits into from Sep 27, 2022

Conversation

kortschak
Copy link
Contributor

This is a collection of fixes that look like are needed. I have some doubts because some fixes here are added in later pages (notable example being the addition of entgql.WithSchemaGenerator() to entc.go in doc/md/tutorial-todo-gql-filter-input.md, which is required to make entc.go successfulliy complete, but which is added in the tutorial in a later page, https://entgo.io/docs/tutorial-todo-gql-schema-generator — though there are others that I've found). My suspicion is that the page order has changed over the construction of the tutorial.

Please take a look.

Without entgql.WithSchemaGenerator entc.go fails to run.
Note that the where:TodoWhereInput parameter is pre-filled in the
generated ent.graphql file.
Only add for the first example to provide a lead to readers.
Copy link
Member

@a8m a8m left a comment

Choose a reason for hiding this comment

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

💯

Appreciate the help, @kortschak.

@a8m a8m merged commit e02622a into ent:master Sep 27, 2022
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.

None yet

2 participants