Skip to content
Snippets Groups Projects

NPS Survey: Use temporary settings

Merged 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

Merged by avatar (May 6, 2025 2:25pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading