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 EDIT role requirement from notification subscriptions #927

Open
jaoxford opened this issue Jan 29, 2025 · 3 comments
Open

Remove EDIT role requirement from notification subscriptions #927

jaoxford opened this issue Jan 29, 2025 · 3 comments

Comments

@jaoxford
Copy link

Hey,

I think we should be able to receive notifications when new scenes from studios / performers are released.

@jaoxford
Copy link
Author

jaoxford commented Jan 29, 2025

I believe changing this line:

updateNotificationSubscriptions(subscriptions: [NotificationEnum!]!): Boolean! @hasRole(role: EDIT)

Could do it, but would there be any side effects or other changes that would need to be done?

@peolic
Copy link
Contributor

peolic commented Jan 29, 2025

Click the notification bell, then "Edit Subscriptions". You can subscribe for both:

  • A new scene from a studio you have favorited.
  • A new scene involving a performer you have favorited.

@jaoxford
Copy link
Author

jaoxford commented Feb 1, 2025

Whilst this is correct, I had to receive the EDIT role from an admin. I think this should be useable by all users.

@DogmaDragon DogmaDragon changed the title I think we should be able to receive notifications for performers we like Remove EDIT role requirement from notification subscriptions Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants