Skip to content
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

chore: add release notes to dashboard page - renderer #9190

Merged
merged 7 commits into from
Oct 3, 2024

Conversation

SoniaSandler
Copy link
Contributor

@SoniaSandler SoniaSandler commented Oct 3, 2024

Signed-off-by: Sonia Sandler [email protected]

What does this PR do?

As part of the dashboard revamp, this PR adds the release notes to the dashboard page. This PR follows #8753, and uses the changes that have been made in the backend side to show the release notes on the dashboard page

  • By default, the release notes banner shows at the top of the dashboard page
  • Clicking the close button will close this banner
  • The update button shows up when an update is available
  • The learn more button sends the user to the full release notes on podman-desktop.io (or GitHub if podman-desktop.io is unavilable )

Screenshot / video of UI

Screenshot from 2024-09-30 12-16-56

If update is unavailable:
Screenshot from 2024-09-30 11-27-10

Notes are unavailable, but GitHub summary is available:
Screenshot from 2024-09-30 12-16-13

Notes are unavailable also on GitHub:
Screenshot from 2024-10-03 10-08-11

What issues does this PR fix or reference?

Related to #8521

How to test this PR?

Open the dashboard page and check if the release notes are visible

  • Tests are covering the bug fix or the new feature

@SoniaSandler SoniaSandler marked this pull request as ready for review October 3, 2024 19:18
@SoniaSandler SoniaSandler requested review from benoitf and a team as code owners October 3, 2024 19:18
@SoniaSandler SoniaSandler requested review from cdrage, axel7083 and gastoner and removed request for a team October 3, 2024 19:18
Copy link
Contributor

@deboer-tim deboer-tim left a comment

Choose a reason for hiding this comment

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

Noted a few nits to clean up, but thanks for the improvements to this in previous PRs.

}
async function getInfoFromNotes() {
let getNotes = await window.podmanDesktopGetReleaseNotes();
notesAvailable = getNotes.releaseNotesAvailable;
Copy link
Contributor

Choose a reason for hiding this comment

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

This makes it seem like the getNotes.releaseNotesAvailable is fairly redundant, you can just use (getNotes.notes != undefined). Not critical now but maybe we can remove it later.

Copy link
Contributor Author

@SoniaSandler SoniaSandler Oct 3, 2024

Choose a reason for hiding this comment

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

I'll change it here and open a separate PR for the main part of it

packages/renderer/src/lib/dashboard/ReleaseNotesBox.svelte Outdated Show resolved Hide resolved
Signed-off-by: Sonia Sandler <[email protected]>
Signed-off-by: Sonia Sandler <[email protected]>
Signed-off-by: Sonia Sandler <[email protected]>
@SoniaSandler SoniaSandler merged commit 12c30c8 into containers:main Oct 3, 2024
15 checks passed
@podman-desktop-bot podman-desktop-bot added this to the 1.13.0 milestone Oct 3, 2024
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.

Make Release Note available from the release version displayed in the status bar
4 participants