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.

Sign up for GitHub

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

Jump to bottom

Check if the user wants to receive mobile notifications #381

Closed
enricostano opened this issue Jun 20, 2018 · 2 comments
Closed

Check if the user wants to receive mobile notifications #381

enricostano opened this issue Jun 20, 2018 · 2 comments
Assignees
@enricostano

Comments

@enricostano
Copy link
Contributor

enricostano commented Jun 20, 2018 •
edited

A push notification can be created only if:

  • the user that would receive it has push_notifications set to true
  • the user is an active member of the organization

Related to #356
@sauloperez
Copy link
Collaborator

sauloperez commented Jun 20, 2018 •
edited

What is this exactly? We want forcefully ask all current users? Just wondering

@enricostano
Copy link
Contributor Author

enricostano commented Jun 20, 2018 •
edited

We just need to filter out the people that set the push_notifications setting to false while creating PushNotification.

@enricostano enricostano mentioned this issue Aug 20, 2018
Merged
@enricostano enricostano self-assigned this Aug 20, 2018
@enricostano enricostano mentioned this issue Aug 20, 2018
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@enricostano enricostano

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@enricostano @sauloperez