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

enh: Allow to set results_delete permission on the frontend #1805

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

susnux
Copy link
Collaborator

@susnux susnux commented Nov 28, 2023

Allow to set the permission for deleting submissions. Also fix the permission check when deleting submission to allow any user with permission to delete a submission instead of restricting to form owner.

@susnux susnux added enhancement New feature or request 3. to review Waiting for reviews feature: 📊 responses & statistics labels Nov 28, 2023
@susnux susnux added this to the 4.0 milestone Nov 28, 2023
@susnux susnux force-pushed the enh/add-ui-for-delete-permission branch from abe1cd1 to d5dd10a Compare November 28, 2023 22:27
Copy link

codecov bot commented Nov 28, 2023

Codecov Report

Merging #1805 (d23e0d7) into main (7a147d9) will increase coverage by 0.77%.
Report is 24 commits behind head on main.
The diff coverage is 100.00%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1805      +/-   ##
============================================
+ Coverage     43.96%   44.73%   +0.77%     
- Complexity      646      649       +3     
============================================
  Files            58       58              
  Lines          2550     2555       +5     
============================================
+ Hits           1121     1143      +22     
+ Misses         1429     1412      -17     

@susnux susnux force-pushed the enh/add-ui-for-delete-permission branch from d5dd10a to 2b27242 Compare November 28, 2023 22:46
@Chartman123 Chartman123 force-pushed the enh/add-ui-for-delete-permission branch from 2b27242 to 93ecf2c Compare November 28, 2023 23:47
@susnux susnux force-pushed the enh/add-ui-for-delete-permission branch from 93ecf2c to e30651f Compare December 9, 2023 14:14
@susnux susnux force-pushed the enh/add-ui-for-delete-permission branch 2 times, most recently from 31c8469 to 6d09699 Compare December 9, 2023 15:59
@Chartman123 Chartman123 force-pushed the enh/add-ui-for-delete-permission branch from 6d09699 to d23e0d7 Compare December 10, 2023 15:41
@Chartman123 Chartman123 merged commit 34096e2 into main Dec 11, 2023
44 checks passed
@Chartman123 Chartman123 deleted the enh/add-ui-for-delete-permission branch December 11, 2023 16:52
@Chartman123
Copy link
Collaborator

/backport to stable3

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Dec 11, 2023
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 backport-request Pending backport by the backport-bot enhancement New feature or request feature: 📊 responses & statistics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants