Skip to content

Commit

Permalink
more doc for reminder
Browse files Browse the repository at this point in the history
  • Loading branch information
Arnaud AMBROSELLI committed Jun 18, 2024
1 parent 00bb141 commit 9cab439
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions STRUCTURATION DES DONNÉES.md
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,8 @@ Nous n'avons pas encore fait de réconciliation de ces données avec le backend,

Les rappels sont des notifications push envoyées à l'utilisateur pour lui rappeler de rentrer ses consos.
Historiquement, c'était une notification locale, gérée localement, mais suite à des bugs sur certains modèles de téléphones, on a décidé de les gérer en backend.
Pour le moment, on a décidé de ne pas permettre à l'utilisateur de mettre en place un rappel sans connexion à internet. En effet, il aura *de toutes façons* besoin d'une connexion pour recevoir la notification push. Donc ça pourrait l'induire en erreur que de lui faire croire qu'il peut jouir de cette fonctionnalité sans connexion.

Ainsi,
- un utilisateur choisit l'heure de son rappel, et le jour de la semaine qu'il préfère
- via un cronjob qui tourne chaque minute, le backend envoie une notification push de rappel à l'heure et au jour choisis
Expand Down
2 changes: 1 addition & 1 deletion api/src/controllers/reminder.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ const toUtcData = ({ timeHours, timeMinutes, daysOfWeek, timezone }) => {
};

router.put(
"/",
["/", "/sync"],
catchErrors(async (req, res) => {
const { matomoId, pushNotifToken, type, timeHours, timeMinutes, id, daysOfWeek, timezone, disabled } = req.body || {};

Expand Down

0 comments on commit 9cab439

Please sign in to comment.