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.

Sign up for GitHub

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

Jump to bottom

Option to disable/hide default mutations #170

Closed
ldesgoui opened this issue Oct 18, 2016 · 1 comment
Closed

Option to disable/hide default mutations #170

ldesgoui opened this issue Oct 18, 2016 · 1 comment
Labels
😌 easy 💅 enhancement 🙏 help-wanted

Comments

@ldesgoui
Copy link

ldesgoui commented Oct 18, 2016 •
edited by calebmer
Loading

Hello,

Currently, the grant/RLS system lets you block all mutations, but they still show in the schema through inspection.
An option to disable those CRUD mutations would be welcome in certain use-cases.

As discussed on gitter, it could simply be implemented with an option passed down to https://github.com/calebmer/postgraphql/blob/3e24703584bb4baaa460da589b072408020a8e1c/src/graphql/schema/getMutationGqlType.ts#L31-L38

Thanks
grncdr added a commit to grncdr/postgraphql that referenced this issue Oct 22, 2016
@grncdr
Add an option to disable default mutations. Closes graphile#170
3b859dd 
This introduces a new CLI flag & option: --disable-default-mutations

When default mutations are disabled, the only mutation operations available
will be those exposed by user defined functions.
@calebmer calebmer mentioned this issue Oct 24, 2016
Merged
calebmer added a commit that referenced this issue Oct 24, 2016
@calebmer
feat(graphql): disable default mutations (#182)
c752260 
Add an option to disable default mutations. Closes #170

All credit goes to @grncdr.

This introduces a new CLI flag & option: --disable-default-mutations

When default mutations are disabled, the only mutation operations available
will be those exposed by user defined functions.

* Use options objects all the way down

This seems more consistent with the rest of the codebase

* pass disableDefaultMutations option through BuildToken

* typo

* Update docs on library usage

* Add integration test for disableDefaultMutations

* Commit updated snapshots

* Revert "Commit updated snapshots"

This reverts commit 6db9086.

* add new schema snapshot
@calebmer
Copy link
Collaborator

calebmer commented Oct 24, 2016 •
edited
Loading

Released in v2.2.0, thanks @grncdr 🎉

Belline pushed a commit to Belline/postgraphql that referenced this issue Dec 18, 2017
@calebmer
feat(graphql): disable default mutations (graphile#182)
1d6b6c9 
Add an option to disable default mutations. Closes graphile#170

All credit goes to @grncdr.

This introduces a new CLI flag & option: --disable-default-mutations

When default mutations are disabled, the only mutation operations available
will be those exposed by user defined functions.

* Use options objects all the way down

This seems more consistent with the rest of the codebase

* pass disableDefaultMutations option through BuildToken

* typo

* Update docs on library usage

* Add integration test for disableDefaultMutations

* Commit updated snapshots

* Revert "Commit updated snapshots"

This reverts commit 6db9086.

* add new schema snapshot
benjie added a commit that referenced this issue Jan 27, 2020
@benjie
feat(errors): Check inflectors give non-empty string (#170)
5e2e697
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
😌 easy 💅 enhancement 🙏 help-wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ldesgoui @calebmer