Skip to content

[stable29] fix(comments): Fix issues thrown by comments sidebar tab code - #44903

Merged
susnux merged 2 commits into
stable29from
backport/44868/stable29
May 28, 2024
Merged

[stable29] fix(comments): Fix issues thrown by comments sidebar tab code#44903
susnux merged 2 commits into
stable29from
backport/44868/stable29

Conversation

@backportbot

@backportbotbackportbotBot commented Apr 17, 2024

Copy link
Copy Markdown

Backport of #44868

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
backportbotBot requested review from Pytal, artonge and susnuxApril 17, 2024 18:48
@backportbotbackportbotBot added bug 3. to review Waiting for reviews labels Apr 17, 2024
@backportbotbackportbotBot added this to the Nextcloud 29 milestone Apr 17, 2024
@susnux
susnux marked this pull request as ready for review April 18, 2024 11:58
@susnux
susnux requested a review from skjnldsv as a code ownerApril 18, 2024 11:58
@AltahrimAltahrim mentioned this pull request Apr 18, 2024
9 tasks
@susnuxsusnux added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 18, 2024
@susnux
susnuxforce-pushed the backport/44868/stable29 branch from 50a6b7e to cf5eac7CompareApril 18, 2024 12:51
@AltahrimAltahrim mentioned this pull request Apr 23, 2024
25 tasks
This was referenced May 15, 2024
@joshtrichards

Copy link
Copy Markdown
Member

Conflicts...

susnux added 2 commits May 28, 2024 10:41
When the comments tab is used instead of the merged activity+comments, then some issues are throws due to prop altering and duplicated names (resourceId as prop and data).
This is fixed as well as some other vue related errors in the sidebar
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/44868/stable29 branch from cf5eac7 to ef7beadCompareMay 28, 2024 08:43
@susnux
susnux enabled auto-merge May 28, 2024 11:34
@susnux
susnux merged commit 782c7ef into stable29May 28, 2024
@susnux
susnux deleted the backport/44868/stable29 branch May 28, 2024 11:53
@AltahrimAltahrim mentioned this pull request May 28, 2024
1 task
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: comments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@joshtrichards@susnux@artonge@Altahrim