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

Not able to create a public share link for recipes of a space the user is not owner of #1998

Closed
kgebert opened this issue Aug 20, 2022 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@kgebert
Copy link
Contributor

kgebert commented Aug 20, 2022

Tandoor Version

1.3.3

Setup

Docker / Docker-Compose

Reverse Proxy

Nginx Proxy Manager (NPM)

Other

No response

Bug description

Me, my sister and my brother are sharing an installation of Tandoor. We have a dedicated recipe space for each and given each other read permissions.
I'm able to publicly share recipes of my own space. Currently I noticed that I'm not able to share recipes of my sister. When clicking Share, I get the following pop-up:

grafik

The public share link is not generated correctly.

  • Either the link should be generated correctly...
  • or - if it is not allowed to share recipes of others - the function should be disabled.

(This could be related to Share link doesn't work between spaces for logged in users #1238).

Relevant logs

No response

@kgebert
Copy link
Contributor Author

kgebert commented Aug 20, 2022

FYI: Just updated to 1.3.3 and this issue is still there.

@vabene1111
Copy link
Collaborator

Interesting, thanks for reporting. I guess that's something I never tested.

In the long term this should probably also be a seperate permission.

@vabene1111 vabene1111 added the bug Something isn't working label Sep 3, 2022
@vabene1111 vabene1111 added this to the 1.3.0 milestone Sep 3, 2022
@vabene1111
Copy link
Collaborator

ok so the problem was actually that you gave each other guest (read only) permission. a guest cannot create a share link (because he is only a guest). I added a proper error message but will rework the permission system in the future to improve this further and make it possible to select those permissions individually

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants