Skip to content

Remove skip confirmation from trial registrations controller

Doug Stull requested to merge 425352-remove-unused-skip-confirmation into master

What does this MR do and why?

  • Remove skip confirmation
  • add in override verbiage for CI checks on methods
  • combine controller spec into request spec
  • add test for incorrect params
  • make resource building more clear in the controller by setting is explicitly and also remove ambiguity on create params around user param being required.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #425352 (closed)

Edited by Doug Stull

Merge request reports