Hide skip all and skip none buttons if skip isn't possible

Description

When a requisition is in the authorized state, and the requisition template doesn't include a skip column – the skip all and skip none links are still visible and functional.

Acceptance Criteria

  • When a requisition with a skip column is in the INITIATED or SUBMITTED state, the "Skip All" and "Skip None" buttons should be visible and funcitonal.

  • If the requisition status is APPROVED, AUTHORIZED, or anything else, the "Skip All" and "Skip None" buttons should be hidden

  • If the requisition doesn't have a skip column, the "Skip All" and "Skip None" buttons should be hidden

Environment

None

Activity

Show:
Mary Jo Kochendorfer
February 20, 2017, 5:40 PM

I apologize for misunderstanding or not reading this ticket thoroughly, but I don't remember us talking about making these into buttons. I thought we agreed to put a space between the links for 3.0. The point of this ticket was to hide these links and the skip column if the requisition template indicated they shouldn't show. That was my understanding for this ticket. I wasn't thinking we were going to change to buttons or anything until after 3.0.

Mary Jo Kochendorfer
February 21, 2017, 1:10 AM
Edited

, Was this bug fixed? Sebastian reset the UAT environment for me today (we have a demo tomorrow so all the new code should be in there... In UAT the All and None links are still displayed when I remove the skip from being displayed. See this screen shot.

Nick Reid
February 21, 2017, 1:35 AM

yea the commit is there – I can't tell you why it's not on test or uat – is your browser caching the page maybe??

Nick Reid
February 21, 2017, 1:42 AM

PS I just did a fresh load of Test and UAT... the button is correct there...

I think you have a caching issue

Mary Jo Kochendorfer
February 21, 2017, 5:36 PM

my confusion with this bug and change to buttons comes from our conversation around because it was my understanding we were only going to add a space between the links as it says in 1808 and that this bug was going to make sure it was hidden when the template deselected the "skip" checkbox. Let's discuss today if we need to do anything or if we leave it as is.

Assignee

Nick Reid

Reporter

Nick Reid

Labels

None

Story Points

1

Components

Sprint

None

Fix versions

Priority

Major
Configure