search

ElixirCLE / pairmotron

Programming Pair Pairifier
https://pairmotron.herokuapp.com
5 stars 4 forks source link

Change pair_retro reflection to be :text type #130

Closed ericworkman closed 7 years ago

ericworkman commented 7 years ago

@mbramson Noticed that the retro reflection text was limited. Found via an error when I tried to write up a longer retro. The default for :string type is varchar(255). This modifies pair_retros to use a :text type in Postgres, so it is now unlimited.

ericworkman commented 7 years ago

@mbramson That comment made me test out roll back. Forgot that I'd have to define down differently, so the rollback failed. This fixes it. Data does not get wiped during migration.

mbramson commented 7 years ago

Awesome! We probably want to add a card to add validate_length(:field, 255) to a bunch of changesets where the :text type isn't as appropriate