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

Notification setting for invites #1944

Closed
1 task
Tracked by #2196
bmarty opened this issue Dec 1, 2023 · 1 comment · Fixed by #1955
Closed
1 task
Tracked by #2196

Notification setting for invites #1944

bmarty opened this issue Dec 1, 2023 · 1 comment · Fixed by #1955
Assignees

Comments

@bmarty
Copy link
Member

bmarty commented Dec 1, 2023

As a user that get invited I want to be able to switch of being notified about invitation so that I can miss them and be sad that I have no one to talk to.

TODO

Existing screen on EXA:

image
  • Add a setting with a toggle in this notifications setting screen to enable / disable the notification for invites, in the Additional settings section which has to be unhidden (currently hidden, since it only contains call setting which is hidden - the call settings stay hidden), with the wording Invitations, and enabled by default (note that we read the value from the SDK, and it should be true by default, that's the default push rules).

For Reference: existing settings on EA

Notification settings Other screen
image image

DOD

When the new setting is disabled, new invitations does not trigger a notification, but are visible in the invites screen.
When the new setting is enable, new invitations trigger a notification, and are visible in the invites screen.

@bmarty
Copy link
Member Author

bmarty commented Dec 1, 2023

Note that there is an Additional settings section which is commented out today:

image

@bmarty bmarty added X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue and removed X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue labels Dec 1, 2023
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

Successfully merging a pull request may close this issue.

1 participant