Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

DEVPROD-920: Conditionally render base status badge #2128

Merged
merged 1 commit into from
Oct 31, 2023
Merged

DEVPROD-920: Conditionally render base status badge #2128

merged 1 commit into from
Oct 31, 2023

Conversation

sophstad
Copy link
Contributor

DEVPROD-920

Description

  • In the tests table, omit the base status badge if undefined. We already do this in the tasks table.

Screenshots

Before:

before

After:
image

@cypress Cypress
Copy link

cypress bot commented Oct 31, 2023

Passing run #13524 ↗︎

0 601 7 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

Only render base status badge if status is defined
Project: Spruce Commit: 80100cc3ac
Status: Passed Duration: 18:43 💡
Started: Oct 31, 2023 4:46 PM Ended: Oct 31, 2023 5:04 PM

Review all test suite changes for PR #2128 ↗︎

@sophstad sophstad requested a review from a team October 31, 2023 17:06
@sophstad sophstad marked this pull request as ready for review October 31, 2023 17:06
khelif96
khelif96 approved these changes Oct 31, 2023
View reviewed changes
Copy link
Contributor

@khelif96 khelif96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Short and sweet :)

@sophstad sophstad merged commit d4e7da5 into evergreen-ci:main Oct 31, 2023
2 checks passed
@sophstad sophstad deleted the DEVPROD-920 branch October 31, 2023 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@SupaJoon SupaJoon SupaJoon approved these changes

@khelif96 khelif96 khelif96 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sophstad @khelif96 @SupaJoon