Skip to content

NPS Survey: Use temporary settings

Warren Gifford requested to merge tr/nps-survey-dismiss-temporary-settings into main

Created by: umpox

Update NPS survey logic to use temporary settings

Migrates all logic around the NPS survey from localStorage to TemporarySettings.

Now the following data is tracked on a 'per-user' basis for authenticated users rather than just a 'per-device' basis:

  • Number of active days for the user
  • Last active date for the user
  • If the user has dismissed the NPS survey
  • If the user has requested to never be shown the NPS survey again.

I've attached a few comments around temporary settings below, would be interested to discuss further!

Merge request reports