Skip to content

Draft: Adopt params.permit (and expect?) more broadly

What does this MR do and why?

Issue #83 recommends using params.permit. I'm submitting this as a draft right now to solicit more feedback.

Further comments on the diffs.

Validation steps

All specs pass locally with this change. I interpret the issue to be a refactor, so the hope is that no specs would have to change throughout, as behavior should be the same.

/cc @leetickett-gitlab

Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading