Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Apr 30, 2025

Backport of #52535

Warning, This backport had conflicts that were resolved with the ours merge strategy and is likely incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

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

@skjnldsv
Copy link
Member

/compile

@skjnldsv skjnldsv marked this pull request as ready for review April 30, 2025 10:22
@skjnldsv skjnldsv force-pushed the backport/52535/stable31 branch from d635865 to b051492 Compare April 30, 2025 10:22
skjnldsv and others added 2 commits April 30, 2025 12:38
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>

[skip ci]
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv skjnldsv force-pushed the backport/52535/stable31 branch from da68e47 to ee90847 Compare April 30, 2025 10:38
class="note-to-recipient"
type="info">
<p v-if="user" class="note-to-recipient__heading">
<p v-if="displayName" class="note-to-recipient__heading">
Copy link
Contributor

Choose a reason for hiding this comment

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

overlooked it in the master PR, but why adding a new computed instead of just:

Suggested change
<p v-if="displayName" class="note-to-recipient__heading">
<p v-if="user.displayName" class="note-to-recipient__heading">

Copy link
Member

Choose a reason for hiding this comment

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

Felt cleaner 🤔

@skjnldsv skjnldsv merged commit 31ab027 into stable31 Apr 30, 2025
203 of 213 checks passed
@skjnldsv skjnldsv deleted the backport/52535/stable31 branch April 30, 2025 16:39
@blizzz blizzz mentioned this pull request May 5, 2025
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants