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

Remove "Expiration Date" Option from link with permission "Only for invited ppl" #9923

Closed
tbsbdr opened this issue Nov 9, 2023 · 0 comments · Fixed by #10083
Closed

Remove "Expiration Date" Option from link with permission "Only for invited ppl" #9923

tbsbdr opened this issue Nov 9, 2023 · 0 comments · Fixed by #10083
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@tbsbdr
Copy link
Contributor

tbsbdr commented Nov 9, 2023

Describe the bug

  • expiration date makes only sense for "public" links or for direct shares with people/groups.

Steps to reproduce

  1. create a link with the role "Only for invited People"
  2. click on 3-dots of the link
  3. option to set an expiration date is available

Expected behavior

  • No option to set an expiration date should be available

Actual behavior

  • option to set an expiration date is available

screenshot_001887

@tbsbdr tbsbdr added Type:Bug Something isn't working Priority:p3-medium Normal priority labels Nov 9, 2023
@JammingBen JammingBen moved this from Qualification to Prio 3 or less in Infinite Scale Team Board Nov 9, 2023
@lookacat lookacat self-assigned this Nov 15, 2023
@tbsbdr tbsbdr changed the title Remove "Expiration Date" Option from private link Remove "Expiration Date" Option from link with permission "Only for invited ppl" Nov 27, 2023
@github-project-automation github-project-automation bot moved this from Prio 3 or less to Done in Infinite Scale Team Board Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants