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

Use nextcloud notifications as time alarm #1129

Open
christianlupus opened this issue Jul 29, 2022 · 3 comments
Open

Use nextcloud notifications as time alarm #1129

christianlupus opened this issue Jul 29, 2022 · 3 comments
Assignees
Labels
API changes involving using the app api Backend Issue or PR related to the backend code enhancement New feature or request Frontend Issue or PR related to the frontend code javascript Pull requests that update Javascript code php Pull requests that update Php code

Comments

@christianlupus
Copy link
Collaborator

christianlupus commented Jul 29, 2022

As discussed in #1064, it might be useful to provide the possibility to use pain nextcloud notifications as a way to notify the user of an alarm. This might not be the most accurate timing but at least there is a notification.

This issue is there primary to keep track of the general idea.


Depends on #1067

@christianlupus christianlupus added API changes involving using the app api Backend Issue or PR related to the backend code Frontend Issue or PR related to the frontend code php Pull requests that update Php code javascript Pull requests that update Javascript code enhancement New feature or request labels Jul 29, 2022
@MarcelRobitaille
Copy link
Collaborator

Is this something that should be configurable in the settings (I am picturing radio buttons for NC notification, browser notification, nothing) or should we always do every kind of notification?

@christianlupus
Copy link
Collaborator Author

I see two ways:

  1. A global per-user setting. There are ringing, NC notifications, and browser notifications. The user can enable/disable each of them separately, potentially depending on the amount of time to be measured (NC notifications are relatively rough, so no fit for <20 min).
  2. The timers could allow the user to individually select the way of notification to be used. Some way of putting checkboxes or so as an additional menu. I do no know a good way to style it ATM.

@github-actions
Copy link

This PR/issue depends on:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API changes involving using the app api Backend Issue or PR related to the backend code enhancement New feature or request Frontend Issue or PR related to the frontend code javascript Pull requests that update Javascript code php Pull requests that update Php code
Projects
None yet
Development

No branches or pull requests

2 participants