New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RUN-2260: Fix: spinner always shown in nextUi jobs list #9045
Conversation
invert spinner loading property semantics hide spinner by default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
it("no content if there is no scm status", async () => { | ||
const wrapper = await mountJobScmStatus({ itemData: {} }); | ||
|
||
let detail = wrapper.findAll("*"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: why you're searching for an asterisk here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
css selector for any element, expecting to find nothing in this case
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
[release/5.2.x] Merge pull request #9045 from rundeck/RUN-2260
Is this a bugfix, or an enhancement? Please describe.
Fix: SCM status spinner is always shown for jobs in the next UI job list screen
Describe the solution you've implemented
Describe alternatives you've considered
Additional context