-
Notifications
You must be signed in to change notification settings - Fork 40
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
if object_name empty for file activity, derive file path from filecache info path #875
Conversation
Codecov Report
@@ Coverage Diff @@
## master #875 +/- ##
=========================================
Coverage 84.12% 84.12%
Complexity 504 504
=========================================
Files 47 47
Lines 1896 1896
=========================================
Hits 1595 1595
Misses 301 301
Continue to review full report at Codecov.
|
05a808a
to
d0356b9
Compare
d0356b9
to
ca8da3a
Compare
ca8da3a
to
611abc4
Compare
@DeepDiver1975 could you re-review ? |
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.
Tested and works
611abc4
to
fd25909
Compare
Rebased to be able to merge |
For some activities, no object_name is stored (for rename, tags, comment, tag). Thus path cannot be derived from object_name and preview are wrongly shown:
This PR fixes it