-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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): add namespace in badge url for apps-in-any-namespace (#15655) #15749
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #15749 +/- ##
=======================================
Coverage 49.66% 49.67%
=======================================
Files 267 267
Lines 46383 46383
=======================================
+ Hits 23036 23039 +3
+ Misses 21086 21084 -2
+ Partials 2261 2260 -1 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Arin <136636751+asapekia@users.noreply.github.com>
Visit this link to see the issue : https://argo-cd.readthedocs.io/en/stable/developer-guide/ci/#public-cd Signed-off-by: PranitRout07 <102309095+PranitRout07@users.noreply.github.com> Signed-off-by: Arin <136636751+asapekia@users.noreply.github.com>
38d9e45
to
6bd369c
Compare
@crenshaw-dev please review. |
Please let me know if any change is required. |
the integration tests passed just by updating the branch. Don't understand why. |
@jannfis Please review this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
UI isn't working/loading with the changes in the PR. |
closing this as the changes are already fixed in PR #16739 |
closes #15655
Checklist: