Skip to content

UI – Update empty Software versions table when installable software present#21118

Merged
jacobshandling merged 4 commits intomainfrom
21053
Aug 7, 2024
Merged

UI – Update empty Software versions table when installable software present#21118
jacobshandling merged 4 commits intomainfrom
21053

Conversation

@jacobshandling
Copy link
Copy Markdown
Contributor

@jacobshandling jacobshandling commented Aug 7, 2024

Addresses 1a of #21053

Screenshot 2024-08-06 at 7 56 38 PM
  • Manual QA for all new/changed functionality

- When no versions, add additional call for software titles available install to know when to display this copy
Copy link
Copy Markdown
Contributor

@ghernandez345 ghernandez345 left a comment

Choose a reason for hiding this comment

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

This looks good. It would be good to try to have some tests around this component as the content on this page the user sees seems quite dynamic. It would be good to have some tests to ensure we are rendering the correct content depending on what the API is sending back and so that we don't break the current functionality in the future if we make more changes to this page.

Feel free to add these in a separate PR. happy to pair together on these also if you'd like

Copy link
Copy Markdown
Member

@RachelElysia RachelElysia left a comment

Choose a reason for hiding this comment

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

Thanks for taking this. Same what Gabe said, we should get tests around this sooner than later, maybe create another ticket on endpoint ops board for transparency.

@jacobshandling jacobshandling merged commit 50ba783 into main Aug 7, 2024
@jacobshandling jacobshandling deleted the 21053 branch August 7, 2024 16:26
@jacobshandling
Copy link
Copy Markdown
Contributor Author

Sounds good, will add tests in follow-up PR – merging to get the release out the door

jacobshandling added a commit that referenced this pull request Aug 7, 2024
…resent (#21118)

## Addresses 1a of #21053 
<img width="1498" alt="Screenshot 2024-08-06 at 7 56 38 PM"
src="https://github.com/user-attachments/assets/d2e97e07-acd7-4f0b-a609-b8b9546a742d">

- [x] Manual QA for all new/changed functionality

---------

Co-authored-by: Jacob Shandling <jacob@fleetdm.com>
jacobshandling added a commit that referenced this pull request Aug 7, 2024
…tware present (#21135)

#### This PR already merged to `main`, see
#21118. This is against the release
branch so it can be included in 4.55.0

Co-authored-by: Jacob Shandling <jacob@fleetdm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants