Skip to content
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

Include federated share activity in sidebar #972

Merged
merged 2 commits into from
Jun 23, 2021
Merged

Include federated share activity in sidebar #972

merged 2 commits into from
Jun 23, 2021

Conversation

JammingBen
Copy link
Contributor

@JammingBen JammingBen commented Jun 23, 2021

Before this fix, the objectId of a received federated share activity was empty. As a result, the activity was not showing in the file sidebar. We now set the objectId by updating the activity as soon as the share has been accepted by the user.

Also fixed a bug where certain activities were not showing in Activities by others.

image

Copy link

@AlexAndBear AlexAndBear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting until owncloud/core#38880 gets merged

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@AlexAndBear AlexAndBear requested a review from jvillafanez June 23, 2021 13:09
@AlexAndBear AlexAndBear self-requested a review June 23, 2021 14:51
@AlexAndBear AlexAndBear merged commit 300fd87 into master Jun 23, 2021
@delete-merged-branch delete-merged-branch bot deleted the issues/970 branch June 23, 2021 14:51
phil-davis added a commit that referenced this pull request Jun 24, 2021
nightly CI had some test failures. It is probably related to the merging of core owncloud/core#38880 and activity #972 yesterday.

This will confirm if there is a test issue here in activity.
This was referenced Jun 24, 2021
@JammingBen JammingBen changed the title Set the objectId for a received federated share activity Include federated share activity in sidebar Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants