NewcastleRSE / dolfin-flutter

The Flutter-based app for the Dolfin Project
0 stars 0 forks source link

12) [CHANGES TO THE USER COMMUNICATION] in the push notifications #184

Closed notimre closed 5 months ago

notimre commented 11 months ago

Goals

The user is more informed about the DOLFIN Trial's time scales through push notifications

Updated behaviour

Current behaviour

The app does not inform the user when the infant has reached 12 months EDD and there are no change to its notifications or functionality. The app does not receive infant EDD from the NPEU’s API

notimre commented 7 months ago

Fix: https://github.com/NewcastleRSE/dolfin-firebase-development/commit/247460d5749eedbb886340cd71c55dfc28511cb5

notimre commented 6 months ago

Madeleine highlighted further problems on the issue:

Quoting her: 12-months EDD (item 12 on the attached scope)

KateCourt commented 6 months ago

There's a few steps that need to be done here:

  1. Currently, the firebase function looks at the NPEU database to get the child's discharge date. This function is called 'doWeHaveDischargeDate'. We need to amend this so that we also look for EDD in addition to discharge date. This then needs saving in the database.
  2. In the firebase function, where we previously checked for 365 days having passed since discharge date, replace with looking at EDD instead.
  3. Add to the push notification for supplement and form reminders function, adding another clause that if there has been 365 days since EDD, the parent should get a push notification with the message requested. You can see how push notifications are created earlier in the function (line 196).

Let's leave Madeleine's final request about the last weekly notification giving an incorrect date until we've fulfilled their initial requirements.

KateCourt commented 5 months ago
KateCourt commented 5 months ago

Create test user to interact with NPEU database,using row 54 https://newcastle.sharepoint.com/:x:/r/sites/rseteam/_layouts/15/Doc.aspx?sourcedoc=%7B84BE7007-8DB7-45D3-8C2A-B6AA7C0D5873%7D&file=DOLFIN%20API%20Test%20Data%20(tadatest)%202022-05-13.xlsx&action=default&mobileredirect=true