-
Notifications
You must be signed in to change notification settings - Fork 751
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
resourceManager bad url using Azure #5817
Conversation
Update bug-report.yml to include v9.13-rc
fix: resourceManager copyUrl
@microsoft-github-policy-service agree |
...nager/ResourceManager.Web/src/components/actions/dnn-action-copy-url/dnn-action-copy-url.tsx
Outdated
Show resolved
Hide resolved
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.
This fix looks good to me. Thanks for the contribution @PabloSR06 🎉
@dnnsoftware/approvers I recommend we go ahead and change this one to target release/9.13.0
so it will be included in this release. Then we can merge release/9.13.0
back into develop
. Thoughts?
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.
Thanks!
Closes #5797
Summary
Add check before copying the url of a file because if the file is in an azure storage account you will get the url of the page and the storage together