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

fix(ui): Change group/grouped visualization argoproj#13565 #14904

Merged

Conversation

vipinachar
Copy link
Contributor

Fixes: #13565

I have added the count of resources on resource tile. Please let me know if it needs ant changes

Demo:-

Uploading argo-cd-grouping.mp4…

@ashutosh16
Copy link
Contributor

@vipinachar would you be able to add a screenshot of ui change?

@vipinachar
Copy link
Contributor Author

oh sorry, I had uploaded a video seems like there was some error.

argo-cd-grouping.mp4

@vipinachar
Copy link
Contributor Author

@ashutosh16 I have uploaded video of my changes

Signed-off-by: Vipin M S <vipinachar2016@gmail.com>
@vipinachar vipinachar force-pushed the fix-ui-change-grouped-visualisation branch from c2314b4 to dbd3ef9 Compare August 7, 2023 09:44
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.90%. Comparing base (05eea87) to head (dbd3ef9).
Report is 48 commits behind head on master.

❗ Current head dbd3ef9 differs from pull request most recent head ab588f7. Consider uploading reports for the commit ab588f7 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #14904      +/-   ##
==========================================
+ Coverage   49.44%   49.90%   +0.45%     
==========================================
  Files         273      262      -11     
  Lines       48662    44983    -3679     
==========================================
- Hits        24063    22447    -1616     
+ Misses      22237    20327    -1910     
+ Partials     2362     2209     -153     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@ishitasequeira ishitasequeira left a comment

Choose a reason for hiding this comment

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

LGTM!!

@vipinachar vipinachar requested a review from a team as a code owner March 24, 2024 01:47
@ishitasequeira ishitasequeira enabled auto-merge (squash) March 24, 2024 01:47
@ishitasequeira ishitasequeira merged commit 37c5f4d into argoproj:master Mar 24, 2024
22 checks passed
lyda pushed a commit to lyda/argo-cd that referenced this pull request Mar 28, 2024
Signed-off-by: Vipin M S <vipinachar2016@gmail.com>
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.com>
Signed-off-by: Kevin Lyda <kevin@lyda.ie>
mkieweg pushed a commit to mkieweg/argo-cd that referenced this pull request Jun 11, 2024
Signed-off-by: Vipin M S <vipinachar2016@gmail.com>
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.com>
Hariharasuthan99 pushed a commit to AmadeusITGroup/argo-cd that referenced this pull request Jun 16, 2024
Signed-off-by: Vipin M S <vipinachar2016@gmail.com>
Co-authored-by: Ishita Sequeira <46771830+ishitasequeira@users.noreply.github.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.

Change group/grouped visualization
3 participants