Created by: Pezmc
Fixes https://github.com/thoughtbot/administrate/issues/1671 Root cause explained on the issue: https://github.com/thoughtbot/administrate/issues/1671#issuecomment-641168322
For the same app as shown in that issue, the time fields now display and update as you'd expect:
You can check that out here: https://github.com/Pezmc/AdministrateTimeBug/tree/bugfix
Tests: Given this is effectively a front-end bug, and there are no Field::Time
's in the example-app (and few tests against that app), I wasn't clear where to add test coverage. I could add a spec/administrate/views/fields/time/_form_spec.rb
that asserts the value is formatted as HH:MM:SS
, but it doesn't feel like that's asserting the right thing.