Skip to content

Refactor manual variables form

Payton Burdette requested to merge 326809-refactor-manual-vars-form into master

What does this MR do and why?

This refactors the manual variables form for triggering a manual job.

  • Use's one set of inputs instead of two sets (no need)
  • Removes technical debt with legacy classes and moves to design system components
  • Improves the unit test coverage
  • Adds feature test to ensure variable values are passed correctly from FE to BE

Screenshots or screen recordings

before after
Screen_Shot_2021-10-25_at_9.42.28_AM Screen_Shot_2021-10-26_at_11.47.17_AM

Manual testing to ensure variable values are passed correctly

Screen_Shot_2021-10-25_at_2.28.21_PM

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 #326809 (closed)

Edited by Payton Burdette

Merge request reports