Feature 2220/stepper review status admin #2270
Conversation
S78901
commented
Apr 26, 2024
S78901
commented
|
@mkimberlin Please review information and the image in the description to verify that this is what you had in mind. |
Apologies. I gave insufficient AC because I was so deeply mired in looking at all these stories together. I have added an AC that clarifies the status values this needs to support. |
…er-review-status-admin
|
@mkimberlin Please review the new specs and let me know if this is what you were envisioning. 
|
Looks perfect. |
| index = i; | ||
| } | ||
| }); | ||
| setActiveIndex(toInteger(index)); |
There was a problem hiding this comment.
Isn't index already an integer?
There was a problem hiding this comment.
Yes, we don't need this anymore.
| const [activeIndex, setActiveIndex] = useState(0); | ||
|
|
||
| useEffect(() => { | ||
| let index = 0; |
There was a problem hiding this comment.
I think there's a shorter way to do this. Maybe something like this:
const index = optionsArr.findIndex(opt => opt.option === revPeriod);There was a problem hiding this comment.
This worked fine, thanks for the suggestion.
