YCPRadioTelescope / RT-Contracts

Back-end architecture for the Radio Telescope Senior Software Design Project for the 2018-2019 academic year
3 stars 1 forks source link

Adding Firebase ID to User column for push notifications #75

Closed lgartrell closed 3 years ago

lgartrell commented 3 years ago

Overview

To support push notifications, we need to add another type to the user.notification_type enum for PUSH_NOTIFICATION. In addition, we need an admin_firebase_id field to the user column --- could be seperate work, but for now will be grouped into this same issue.

Possible plan of work

  1. investigate user.notifcation_type enum
  2. add PUSH_NOTIFICATION to the end of the enum list
  3. investigate the user table
  4. add admin_firebase_id field to the user column -- this should be defaulted to null
  5. update user schema to represent new changes (schema is on the shared drive)
  6. make sure the tests all still pass, and any schema representing the user table is updated
trevorswann commented 3 years ago

Added PUSHNOTIFICATION to the end of the enum, and added firebase_id to the user table. Having issue with the variable type with the enum list which needs to be fixed, then a couple things may need to be added to test. Updating schema now.