-
Notifications
You must be signed in to change notification settings - Fork 32
🐛 Fix listing (currently, project activity only shows the current user's actions; activities of other users are not visible) #7732
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 listing (currently, project activity only shows the current user's actions; activities of other users are not visible) #7732
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7732 +/- ##
===========================================
- Coverage 87.37% 69.24% -18.13%
===========================================
Files 1838 700 -1138
Lines 71325 33274 -38051
Branches 1202 174 -1028
===========================================
- Hits 62318 23041 -39277
- Misses 8677 10175 +1498
+ Partials 330 58 -272
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
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.
- you should add a test that hits this condition. That is why you did not detect it last time!
GitHK
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.
I'd also appreciate a test to go with this change
❌ Sorry but I didn't understand the command. Please consult the commands documentation 📚. |
|
|
@Mergifyio queue |
🟠 Waiting for conditions to match
|
78abf4b
into
ITISFoundation:master



What do these changes do?
Related issue/s
How to test
Dev-ops