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

Hide linked file actions from users who have access to results view #1920

Merged

Conversation

Koc
Copy link
Collaborator

@Koc Koc commented Jan 27, 2024

Use case:

  1. Share Form with User, give him access to view results

Current behavior

  • User can see Link file/Unlink file/Open file buttons (but he not allowed to perform this actions due to checks on BE side)

Behavior after fix

  • Only Form owner can see mentioned buttons
Screenshots

image
image

@Koc
Copy link
Collaborator Author

Koc commented Jan 27, 2024

Test failures looks unrelated

@susnux
Copy link
Collaborator

susnux commented Jan 28, 2024

Not sure if we need a new permission or if it should be within the PERMISSION_EDIT

@Koc Koc force-pushed the feature/add-permission-for-linking-file branch 2 times, most recently from 591a0eb to 1e4ca25 Compare January 29, 2024 16:36
@Koc
Copy link
Collaborator Author

Koc commented Jan 29, 2024

@susnux I've updated PR with reusing already existent PERMISSION_EDIT permission instead of introduction of the new one

@susnux susnux added bug Something isn't working 3. to review Waiting for reviews labels Jan 29, 2024
@Koc Koc force-pushed the feature/add-permission-for-linking-file branch from 1e4ca25 to e1cfbc7 Compare January 29, 2024 22:50
Copy link

codecov bot commented Jan 29, 2024

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (0402c08) 44.79% compared to head (4396fc3) 44.75%.
Report is 14 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1920      +/-   ##
============================================
- Coverage     44.79%   44.75%   -0.04%     
- Complexity      686      687       +1     
============================================
  Files            59       59              
  Lines          2697     2699       +2     
============================================
  Hits           1208     1208              
- Misses         1489     1491       +2     

@Chartman123 Chartman123 mentioned this pull request Jan 31, 2024
4 tasks
@Koc Koc force-pushed the feature/add-permission-for-linking-file branch from e1cfbc7 to 1abd1fd Compare January 31, 2024 23:35
@Chartman123
Copy link
Collaborator

Ok, then just do a rebase and squash the commits into one :)

Signed-off-by: Konstantin Myakshin <molodchick@gmail.com>
@Koc Koc force-pushed the feature/add-permission-for-linking-file branch from 1abd1fd to 4396fc3 Compare February 1, 2024 13:04
@Chartman123 Chartman123 merged commit 51e4ca1 into nextcloud:main Feb 1, 2024
48 of 50 checks passed
@Koc Koc deleted the feature/add-permission-for-linking-file branch February 1, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants