Skip to content

[stable30] fix(files_sharing): Show note if note exists on share - #52850

Merged
nfebe merged 3 commits into
stable30from
backport/52813/stable30
May 26, 2025
Merged

[stable30] fix(files_sharing): Show note if note exists on share#52850
nfebe merged 3 commits into
stable30from
backport/52813/stable30

Conversation

@backportbot

@backportbotbackportbotBot commented May 15, 2025

Copy link
Copy Markdown

Backport of #52813

Warning, This backport's changes differ from the original and might be 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.

@backportbotbackportbotBot added this to the Nextcloud 30.0.11 milestone May 15, 2025
@nfebe
nfebeforce-pushed the backport/52813/stable30 branch from 4179226 to 7a7aa1dCompareMay 15, 2025 06:02
@nfebe
nfebe marked this pull request as ready for review May 15, 2025 06:03
@nfebe
nfebe enabled auto-merge May 15, 2025 06:03
@nfebe

Copy link
Copy Markdown
Contributor

/compile

@nfebe

Copy link
Copy Markdown
Contributor

/backport to stable29

@skjnldsvskjnldsv mentioned this pull request May 15, 2025
@nextcloud-botnextcloud-bot mentioned this pull request May 15, 2025
20 tasks
@AndyScherzinger
AndyScherzingerforce-pushed the backport/52813/stable30 branch from 18c54c0 to 8db4e54CompareMay 26, 2025 09:22
nfebe added 2 commits May 26, 2025 18:39
When a share includes a note, ensure the note checkbox is checked and the note is
displayed when reopening the share sidebar.
Fixes: #52794.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
If a share note is present, the advanced settings should expand automatically,
without the user needing to click.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@AndyScherzinger
AndyScherzingerforce-pushed the backport/52813/stable30 branch from 8db4e54 to 28ae3d3CompareMay 26, 2025 16:39
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile /

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nfebe
nfebe merged commit 58bdf17 into stable30May 26, 2025
@nfebe
nfebe deleted the backport/52813/stable30 branch May 26, 2025 17:11
@backportbot

Copy link
Copy Markdown
Author

The backport to stable29 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable29
git pull origin stable29
# Create the new backport branch
git checkout -b backport/52850/stable29
# Cherry pick the change from the commit sha1 of the change against the default branch# This might cause conflicts, resolve them
git cherry-pick bd467605 28ae3d30 b7aa85dc
# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/52850/stable29

Error: Failed to check for changes with origin/stable29: No changes found in backport branch


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

@nfebe

Copy link
Copy Markdown
Contributor

/backport to stable29

@backportbot

Copy link
Copy Markdown
Author

The backport to stable29 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable29
git pull origin stable29
# Create the new backport branch
git checkout -b backport/52850/stable29
# Cherry pick the change from the commit sha1 of the change against the default branch# This might cause conflicts, resolve them
git cherry-pick bd467605 28ae3d30 b7aa85dc
# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/52850/stable29

Error: Failed to check for changes with origin/stable29: No changes found in backport branch


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

@nextcloud-botnextcloud-bot mentioned this pull request Jun 4, 2025
11 tasks
@nextcloud-botnextcloud-bot mentioned this pull request Sep 18, 2025
@nextcloud-botnextcloud-bot mentioned this pull request Sep 25, 2025
2 tasks
@blizzzblizzz mentioned this pull request Oct 14, 2025
Sign up for freeto 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.

5 participants

@nfebe@AndyScherzinger@artonge@skjnldsv@nextcloud-command