Skip to content
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

[full-ci] Access right sidebar panels via URL #8021

Merged
merged 3 commits into from
Nov 30, 2022
Merged

Conversation

JammingBen
Copy link
Collaborator

@JammingBen JammingBen commented Nov 24, 2022

Description

Opening the right sidebar (including its panels) is now possible via URL param.

For private or internal links it only requires the new details param in the URL. For other URLs (e.g. personal space, project space) the scrollTo param including the resource id is needed as well.

The following values can be used for the details param:

  • details - sidebar open, no specific panel
  • actions - actions panel
  • sharing - share panel
  • versions - versions panel
  • space-share - members panel (project space only)

Note: "-item"-prefix for sidebar panels has been removed in the course of this.

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

@JammingBen JammingBen self-assigned this Nov 24, 2022
Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

Apparently we have the suffix -item in all panel names... IMO we should kill that 😆 Could you check if that's easily possible? And if not, then please omit the -item suffix from the query param and append it in the useScrollTo composable where you publish the event.

@ownclouders
Copy link
Contributor

ownclouders commented Nov 25, 2022

Results for oC10Files3 https://drone.owncloud.com/owncloud/web/30109/19/1

💥 The acceptance tests failed on retry. Please find the screenshots inside ...

webUIFilesDetails-fileDetails_feature-L32.png

webUIFilesDetails-fileDetails_feature-L32.png

webUIFilesDetails-fileDetails_feature-L70.png

webUIFilesDetails-fileDetails_feature-L70.png

@JammingBen JammingBen changed the title Access right sidebar panels via URL [full-ci] Access right sidebar panels via URL Nov 25, 2022
@JammingBen JammingBen force-pushed the sidebar-panels-via-url branch 2 times, most recently from 3739c28 to 845916b Compare November 25, 2022 09:32
@JammingBen JammingBen marked this pull request as ready for review November 25, 2022 09:32
@JammingBen JammingBen added the Status:Needs-Review Needs review from a maintainer label Nov 25, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 25, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

72.6% 72.6% Coverage
0.0% 0.0% Duplication

Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

Awesome! 🥳

@michaelstingl please see the changelog item for available panel names for the details query item for the private link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Needs-Review Needs review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Open Sidebar panels with parameter added to link
3 participants