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

feat: add css to support external custom style #13279

Merged
merged 3 commits into from
Apr 28, 2023

Conversation

ashutosh16
Copy link
Contributor

@ashutosh16 ashutosh16 commented Apr 18, 2023

This change intended to restrict banners only to applications and not to the entire argocd. With the help of custom-styles, you can customize the look and feel of the components. The change, however, includes adding banners only when the application name contains one of the following characters: prd, prod, or production

Screen.Recording.2023-04-18.at.9.53.50.AM.mov

@ashutosh16 ashutosh16 force-pushed the custom-styles-appbanner branch 4 times, most recently from b241af9 to bc65cc1 Compare April 20, 2023 20:00
@codecov
Copy link

codecov bot commented Apr 20, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (03513eb) 49.06% compared to head (41a5e95) 49.06%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #13279      +/-   ##
==========================================
- Coverage   49.06%   49.06%   -0.01%     
==========================================
  Files         248      248              
  Lines       42816    42816              
==========================================
- Hits        21009    21006       -3     
- Misses      19688    19690       +2     
- Partials     2119     2120       +1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ashutosh16 ashutosh16 marked this pull request as ready for review April 24, 2023 16:16
@ashutosh16 ashutosh16 force-pushed the custom-styles-appbanner branch 7 times, most recently from 88dd302 to 9cf6eee Compare April 27, 2023 17:11
@crenshaw-dev crenshaw-dev enabled auto-merge (squash) April 27, 2023 17:31
auto-merge was automatically disabled April 27, 2023 19:08

Head branch was pushed to by a user without write access

@ashutosh16 ashutosh16 force-pushed the custom-styles-appbanner branch 3 times, most recently from ccc3006 to 99be4ec Compare April 27, 2023 19:57
Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>
Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>
@crenshaw-dev crenshaw-dev merged commit b059d78 into argoproj:master Apr 28, 2023
24 checks passed
@ashutosh16 ashutosh16 deleted the custom-styles-appbanner branch May 1, 2023 16:11
@ashutosh16
Copy link
Contributor Author

@crenshaw-dev Can we cherry pick this in 2.7?

@crenshaw-dev
Copy link
Collaborator

@leoluz leoluz added the cherry-pick/2.7 Candidate for cherry picking into the 2.7 release branch label May 8, 2023
@crenshaw-dev
Copy link
Collaborator

/cherry-pick release-2.7

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request May 25, 2023
* feat: add external css to customize banner

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>

* fix lint

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>

---------

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>
crenshaw-dev pushed a commit that referenced this pull request May 25, 2023
* feat: add external css to customize banner



* fix lint



---------

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>
Co-authored-by: asingh <11219262+ashutosh16@users.noreply.github.com>
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…proj#13746)

* feat: add external css to customize banner

* fix lint

---------

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>
Co-authored-by: asingh <11219262+ashutosh16@users.noreply.github.com>
Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
yyzxw pushed a commit to yyzxw/argo-cd that referenced this pull request Aug 9, 2023
* feat: add external css to customize banner

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>

* fix lint

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>

---------

Signed-off-by: ashutosh16 <11219262+ashutosh16@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick/2.7 Candidate for cherry picking into the 2.7 release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants