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

Revert custom queries and mutations #80

Merged
merged 2 commits into from
Jul 12, 2024
Merged

Conversation

j-x-han
Copy link
Collaborator

@j-x-han j-x-han commented Jul 12, 2024

Follow up to #77

Removes the code for custom queries and mutations from test_app, since they're not relevant as defaults for a starting point. The code for adding custom queries/mutations is also already contained in the doc, which this PR does not remove.

@j-x-han j-x-han requested review from jgadling and ninabernick July 12, 2024 21:16
@@ -1,17 +0,0 @@
import strawberry
Copy link
Collaborator Author

@j-x-han j-x-han Jul 12, 2024

Choose a reason for hiding this comment

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

This PR currently removes custom_mutations.py and custom_queries.py entirely, but I could see it being helpful to leave the files there as templates and just removing the mutations/queries (or commenting them out). LMK which is preferred!

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah I think we could have an examples folder for that kind of thing?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

sgtm, I'll merge this one as-is for now and we can set up examples in a follow up 😄

@j-x-han j-x-han merged commit 1373a23 into main Jul 12, 2024
3 checks passed
@j-x-han j-x-han deleted the j-x-han/revert-custom-queries branch July 12, 2024 21:31
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