Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove email from surveyors table #595

Open
lfilmeyer opened this issue Aug 12, 2024 · 0 comments
Open

Remove email from surveyors table #595

lfilmeyer opened this issue Aug 12, 2024 · 0 comments
Labels
Priority: Low Role: ETL/Backend Tech Debt Fixing this will save us time long-term

Comments

@lfilmeyer
Copy link
Contributor

The email field on surveyors is a duplicate of the one on users and can be removed. We maybe once had a need for both (ex. login email vs. contact email) but we've decided to simplify it to just use 1 email on users.

A question I have regarding removing this field - how does the deploy process work for this app? For previous applications I've worked on, removing a field took multiple deploys to avoid errors & downtime. Not that we have to worry about downtime much for this app yet, but it does prompt questions like "how do migrations get run?"

@lfilmeyer lfilmeyer added Role: ETL/Backend Tech Debt Fixing this will save us time long-term Priority: Low labels Aug 12, 2024
@lfilmeyer lfilmeyer added this to the Post-MVP Launch milestone Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low Role: ETL/Backend Tech Debt Fixing this will save us time long-term
Projects
None yet
Development

No branches or pull requests

1 participant