Closed jpuzz0 closed 1 week ago
/retest
This is a nice solution, but I do have another suggestion. I feel we could avoid inferring from the pathname, instead, we can pass
isRedirectingToDetailsPage
or something like this into the modal, and only when it's set to true, navigate to the details page... And we can default it to true and set it only to false when importing on the create run/schedules page... Matching with the pathname is not always secure because if we change the pathname in the future, it's hard to find the issue here. This is OK for now, but just a suggestion.
That's fair, I don't mind either way, so I made the change to add a prop instead.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.97%. Comparing base (
a0d40fd
) to head (0b7abcc
). Report is 10 commits behind head on main.
@jpuzz0 we also should prevent navigation when importing from the empty states bc it is pretty much the same page anyway
we also should prevent navigation when importing from the empty states bc it is pretty much the same page anyway
@Gkrumbach07 So inside of GlobalNoPipelines
, prevent the redirect there? I'm not sure how this would be different from the redirect that we want from the pipelines table action. Maybe I'm missing something here though.
/approve
was mis reading what was changed
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: DaoDaoNoCode, Gkrumbach07
The full list of commands accepted by this bot can be found here.
The pull request process is described here
https://issues.redhat.com/browse/RHOAIENG-11326
Description
Add condition to pipeline import modal to only redirect when not located in the
/create
path associated with runs/schedulesDemo: https://github.com/user-attachments/assets/28f10956-9b84-465e-8db6-bf9fda2d5f6d
How Has This Been Tested?
Test Impact
No new tests added
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
main