Code review comment for lp:~stevanr/lava-dashboard/image-report-editor-frontend

Revision history for this message
Antonio Terceiro (terceiro) wrote :

On Fri, Sep 13, 2013 at 10:19:41AM -0000, Stevan Radaković wrote:
> Of course you can, the only thing you have to do is something like:
> "lava-server manage migrate dashboard_app 0029" before the migration.

Sure, but that defeats the point of using migrations since it will not
require specific manual steps on a specific upgrade.

> I realize our scripts on staging and production are not that
> sophisticated, so I reverted the deleted migration and created new one
> based on that migration step and my changes to the model.

The point of migrations is to have your database evolve without manual
intervention other than "upgrade my database", and having this "upgrade
my datavase" action always to the right thing.

 review approve

review: Approve

« Back to merge proposal