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: space notification links #10603
base: stable-8.0
Are you sure you want to change the base?
Conversation
Fixes links for incoming space notifications if the user has been added to a new space. Also makes sure links are being removed if a user has been removed from a space and the space had been loaded before.
@@ -175,21 +176,47 @@ export default { | |||
const space = store.getters['runtime/spaces/spaces'].find( | |||
(s) => s.fileId === messageRichParameters?.space?.id.split('!')[0] && !s.disabled | |||
) | |||
if (space) { | |||
|
|||
if (space && subject === 'Removed from Space') { |
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.
Isn't the subject translated by the backend? So this would only work for users who have set English as language?
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.
also... don't we have a different property to check on? Subject is really ugly for this :-(
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.
Haha yes, totally forgot 🙈 Well, then this is blocked by the server. There is no other way to distinguish between the subjects.
I'll create an issue for it.
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.
thank you 💪
Quality Gate passedIssues Measures |
Currently blocked by owncloud/ocis#8645. |
Description
Fixes links for incoming space notifications if the user has been added to a new space. Also makes sure links are being removed if a user has been removed from a space and the space had been loaded before.
Related Issue
Types of changes