Skip to content

fix(PageInfoBar): Do not show versionsLink for public shares#2305

Open
pymnh wants to merge 2 commits intonextcloud:mainfrom
pymnh:fix/do-not-show-version-link-on-public-share
Open

fix(PageInfoBar): Do not show versionsLink for public shares#2305
pymnh wants to merge 2 commits intonextcloud:mainfrom
pymnh:fix/do-not-show-version-link-on-public-share

Conversation

@pymnh
Copy link

@pymnh pymnh commented Feb 28, 2026

📝 Summary

  • Last edited information should not be a link if I can not open the versions sidebar
  • Beside checking for canEdit, should also check if !isPublic (public shares have no versions sidebar)

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

Fix in 2430112 was incomplete,
should check if can edit **and** if is not a public share.

Signed-off-by: Peymaneh <peymaneh@posteo.net>
Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

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

Thanks a lot, makes sense indeed! Just one comment, otherwise looks good.

Signed-off-by: Peymaneh <peymaneh@posteo.net>
@pymnh pymnh force-pushed the fix/do-not-show-version-link-on-public-share branch from a234097 to 4a53522 Compare February 28, 2026 23:54
@pymnh pymnh requested a review from mejo- February 28, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants