/h

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

Set a server-side default for the NIPSA column #2272

Closed
wants to merge 1 commit into
base: master
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@nickstenning @landscape-bot @seanh @tilgovi
@nickstenning
Contributor

nickstenning commented Jun 3, 2015

This should really be done for all migrations that involve adding a non-nullable column. This makes it possible to run the migration in staging without breaking production.

You will need to throw away your development database (or do something fiddly like downgrade to 4a97f680ecca first and then upgrade to 29d0200ba8a9).
@nickstenning
Set a server-side default for the NIPSA column 
This should really be done for all migrations that involve adding a
non-nullable column. This makes it possible to run the migration in
staging without breaking production.

You will need to throw away your development database (or do something
fiddly like downgrade to 4a97f680ecca first and then upgrade to
29d0200ba8a9).
Loading status checks…
04461ff
@landscape-bot

This comment has been minimized.

Show comment
Hide comment
@landscape-bot

landscape-bot Jun 3, 2015

Code Health
Code quality remained the same when pulling 04461ff on hypothesis:fix-nipsa-migration into 07b7941 on hypothesis:master.

landscape-bot commented Jun 3, 2015

Code Health
Code quality remained the same when pulling 04461ff on hypothesis:fix-nipsa-migration into 07b7941 on hypothesis:master.
@seanh

This comment has been minimized.

Show comment
Hide comment
@seanh

seanh Jun 3, 2015

Contributor

Merged
Contributor

seanh commented Jun 3, 2015

Merged

@seanh seanh closed this Jun 3, 2015

@tilgovi tilgovi deleted the fix-nipsa-migration branch Jun 3, 2015

@tilgovi

This comment has been minimized.

Show comment
Hide comment
@tilgovi

tilgovi Jun 3, 2015

Contributor

Thanks, @nickstenning
Contributor

tilgovi commented Jun 3, 2015

Thanks, @nickstenning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment