widefix / actual_db_schema

Wipe out inconsistent DB and schema.rb when switching branches. To do that, install this gem and use the standard rake db:migrate command.
https://blog.widefix.com/actual-db-schema/
MIT License
300 stars 7 forks source link

Add UI #70

Closed VladislavSokov closed 3 months ago

VladislavSokov commented 4 months ago
VladislavSokov commented 4 months ago

Screenshot from 2024-06-26 19-07-54 Screenshot from 2024-06-26 19-07-44

ka8725 commented 4 months ago

Manual testing results:

  1. down migration cannot be rolled back, so hide the button?

    image
  2. If no phantom migrations the table looks clunky

    image

    Show some text saying no phantom migrations instead of the empty table? Or fix the table look so it's clear what's going on.

  3. WDYT if we don't show down phantom migrations in the table at all? Reason - there is no point in running it up and also since it's down the user cannot do with it anything. This kind of migration is useless for the current branch too. So if it's hidden the user won't notice. It's not a relevant thing that can be destroyed. I tend to think we can remove it entirely on down. Please tell what do you think, thanks!

VladislavSokov commented 4 months ago

3. WDYT if we don't show down phantom migrations in the table at all? Reason - there is no point in running it up and also since it's down the user cannot do with it anything. This kind of migration is useless for the current branch too. So if it's hidden the user won't notice. It's not a relevant thing that can be destroyed. I tend to think we can remove it entirely on down. Please tell what do you think, thanks!

I think this is a great idea