-
Notifications
You must be signed in to change notification settings - Fork 32
🎨Storage: display path field is url encoded by parts #7285
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
🎨Storage: display path field is url encoded by parts #7285
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7285 +/- ##
===========================================
+ Coverage 74.72% 88.40% +13.68%
===========================================
Files 1055 1351 +296
Lines 45265 55947 +10682
Branches 227 481 +254
===========================================
+ Hits 33823 49461 +15638
+ Misses 11372 6354 -5018
- Partials 70 132 +62
Continue to review full report in Codecov by Sentry.
|
pcrespov
left a comment
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.
thx
odeimaiz
left a comment
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.
Merci!
14e668e to
95eb0d0
Compare
95eb0d0 to
820b7d7
Compare
|



What do these changes do?
This PR brings url encoding (including
/characters) for thedisplay_pathfield inPathMetaDataobject.This is needed as project names and node names may contain any kind of characters including
/characters.Related issue/s
How to test
Dev-ops checklist