-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Improve TagURL #6054
Improve TagURL #6054
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
d8db14a
to
88bb875
Compare
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
I've found an issue in this PR and I reported it here @blackdevelopa |
88bb875
to
d32fa70
Compare
cbe3301
to
38f1488
Compare
Kudos, SonarCloud Quality Gate passed! |
@blackdevelopa I could repro the issue on prod and I cannot repro the issue in the PR. Meaning the issue seems to be fixed 💯 Adding QA Pass and can be merged whenever we get the 2 approvals (seems they were reset) |
Development & PR Process
release-xx
label to identify the PR slated for a upcoming release (will be used in release discussion)needs-dev-review
label when work is completedneeds-qa
label when dev review is completedQA Passed
label when QA has signed offDescription
This PR updates tag url to origin instead of active tab url.
Screenshots/Recordings
If applicable, add screenshots and/or recordings to visualize the before and after of your change
Issue
Progresses #681
Checklist