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

Add a button to download upgradable packages as CSV #3098

Merged
merged 16 commits into from
Nov 7, 2024

Conversation

janvhs
Copy link
Member

@janvhs janvhs commented Oct 23, 2024

Description

This commit adds a CSV download button to include all patches to the UpgradablePackages view.

Preview

image

How was this tested

  • Added test for UpgradablePackages
  • Added UpgradablePackages empty story

What is open ?

  • Missing e2e test, maybe in follow up pr

@janvhs janvhs requested a review from dottorblaster October 23, 2024 08:29
@janvhs janvhs added javascript Pull requests that update Javascript code env Create an ephimeral environment for the pr branch labels Oct 23, 2024
@janvhs janvhs requested a review from EMaksy October 24, 2024 12:55
@janvhs janvhs force-pushed the csv-in-upgradable-packages branch from 844249e to 9d60434 Compare October 24, 2024 12:56
@janvhs janvhs marked this pull request as ready for review October 24, 2024 13:04
@EMaksy EMaksy force-pushed the csv-in-upgradable-packages branch from 1683d5a to b731206 Compare November 6, 2024 11:25
@EMaksy EMaksy self-assigned this Nov 6, 2024
@EMaksy EMaksy requested a review from arbulu89 November 6, 2024 11:43
Copy link
Member

@EMaksy EMaksy left a comment

Choose a reason for hiding this comment

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

Just for the future, left a comment

render: (content, _) => <div className="font-bold">{content}</div>,
},
{
title: 'Latest Package',
key: 'latestPackage',
key: 'latest_package',
Copy link
Member

Choose a reason for hiding this comment

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

Wondering with which naming convention we want to stick on this ?
As we do it different through the project xD
@janvhs , @dottorblaster ideas ?

@EMaksy EMaksy added the bug Something isn't working label Nov 6, 2024
@EMaksy EMaksy changed the title Fix CSV Download to Include All Patches Add a button to download upgradable packages as CSV Nov 6, 2024
Copy link
Contributor

@arbulu89 arbulu89 left a comment

Choose a reason for hiding this comment

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

@EMaksy Some comments

Copy link
Contributor

@dottorblaster dottorblaster left a comment

Choose a reason for hiding this comment

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

I wanted to ask @janvhs to provide one more test but you @EMaksy did it before 😄

Besides what @arbulu89 said (great comments BTW, really nothing to add), just a tiny suggestion on my side

@EMaksy EMaksy force-pushed the csv-in-upgradable-packages branch from 56a0891 to c704f16 Compare November 7, 2024 08:18
Copy link
Contributor

@arbulu89 arbulu89 left a comment

Choose a reason for hiding this comment

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

LGTM!

@EMaksy EMaksy merged commit c045e4a into main Nov 7, 2024
30 checks passed
@EMaksy EMaksy deleted the csv-in-upgradable-packages branch November 7, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working env Create an ephimeral environment for the pr branch javascript Pull requests that update Javascript code
Development

Successfully merging this pull request may close these issues.

4 participants