Closed Bug 1065650 Opened 7 years ago Closed 7 years ago

"This field is required" error trying to create a new job exclusion, even though all fields are filled

Categories

(Tree Management :: Treeherder, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: mdoglio)

References

Details

Attachments

(2 files)

Attached image screenshot
I'm trying to create a new job exclusion profile per the attached screenshot, but am getting a "This field is required" error when I hit save without any indication of what's missing.
Blocks: 1031857
Priority: -- → P1
Assignee: nobody → mdoglio
Attachment #8487625 - Flags: review?(jeads)
Status: NEW → ASSIGNED
Making not a blocker since this has turned out to be a UX issue and it's still possible to proceed with bug 1031857 in the meantime.
Priority: P1 → P2
Commit pushed to master at https://github.com/mozilla/treeherder-ui

https://github.com/mozilla/treeherder-ui/commit/35fdc5bdcee1ddc3d5dc8b202bb43d39c5841e02
Bug 1065650 - make job exclusion form validation more explicit

Also:
Move the exclusion name and description to the top of the form to
make it more prominent.
Change the max-height of the menu bar panels to avoid long scrolling.
Fix sort order in the job exclusions options
Add a back button to the job exclusion form
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attachment #8487625 - Flags: review?(jeads)
Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/70dfc781ed382251c6c8a6581e84765922f975a8
Bug 1065650 - make job exclusion form validation more explicit

Also:
Move the exclusion name and description to the top of the form to
make it more prominent.
Change the max-height of the menu bar panels to avoid long scrolling.
Fix sort order in the job exclusions options
Add a back button to the job exclusion form
You need to log in before you can comment on or make changes to this bug.