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

Add support for .bypassrules directive #228

Closed
wants to merge 5 commits into from

Conversation

sirtwist
Copy link

@sirtwist sirtwist commented Feb 7, 2021

These changes add support for .bypassrules=true directive, to allow for bypassing rules but not requiring impersonation. An example of a need for this would be for updating fields where there are read-only rules in place, such as for calculating a value based on other fields where you don't want users to be able to update the calculated value.

Closes #83.

@giuliov
Copy link
Member

giuliov commented Feb 8, 2021

Thanks for the contribution: it will take some time before I merge and add the feature documentation at aggregator-docs.

@giuliov
Copy link
Member

giuliov commented May 1, 2021

Merged in release/1.1 branch

@giuliov giuliov closed this May 1, 2021
@giuliov giuliov mentioned this pull request Jun 12, 2021
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.

New directive : bypassrule
2 participants