Closed DevinCLane closed 1 year ago
@DevinCLane do you need re-review, or is there anything we can do to help with this PR?
I'm not feeling well, so if this is blocking feel free to have someone else take it over! A re-review could be good in the meantime?
Closes #78
Description
Pretty bigly PR. Does a bunch of things:
Backend stuff
flags
table to allow setting of flags on users. Used in this case to flag whether a user has skipped onboardingGET /api/auth/current-user
which returns the logged-in user and flagsPUT /api/user/onboarding/skip
which upserts the skipped onboarding flag for the logged-in userPOST /api/auth/login
returns flags as wellprofile
to the User model that will get us the public, client-friendly object, so we don't have to call_.pick
every time we want to send that data alongTesting:
truncateDatabase
function as this is an antipattern and we should generally not allow a single test to impact other testslocalhost:3000
in all of thecy.visits
Frontend stuff:
Testing
Manually by clearing the DB, logging in, clicking "skip onboarding", then logging out, logging in again, and it should put you right on
/directory
and not the onboarding pagePlus server tests
Type of change
Screenshots
Please include any screenshots if there are visual changes.
Checklist: