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

Added check for unallowed action parameter #15

Merged

kaitlyn-sharo merged 1 commit into main from 181-check-parameters

Jan 8, 2024

Contributor

kaitlyn-sharo commented Jan 5, 2024

No description provided.


          added check for unallowed action parameter

8cd283c

kaitlyn-sharo requested review from lawerencem, brianpippin, nextcen-dgemoets, jaudick and dereknop

January 5, 2024 18:52

nextcen-dgemoets approved these changes

View reviewed changes

Contributor

nextcen-dgemoets left a comment

It occurs to me that we don't check for duplicate keys in the parameters field, but I can live with that if you can. :)

Base automatically changed from 172-more-rules to main

January 8, 2024 21:00

brianpippin approved these changes

View reviewed changes

kaitlyn-sharo merged commit 8f779bd into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

nextcen-dgemoets nextcen-dgemoets approved these changes

brianpippin brianpippin approved these changes

lawerencem Awaiting requested review from lawerencem

jaudick Awaiting requested review from jaudick

dereknop Awaiting requested review from dereknop

Labels

None yet