created
elliotcm
username_0
I've updated the schema file, but there's a couple of test failures and a giant sack of deprecations to fix up and I'm out of allotted time.

We'll create a ticket for this.
comment
created
elliotcm
username_0
https://trello.com/c/HmaPc3Sg/371-complete-whitehall-rails-upgrade-to-51
comment
created
cbaines
username_1
I've pushed a commit fixing some ordering issues with `has_many` things. Still need to try and understand the issue more.

I found something about the `_routes` issue, but unfortunately it doesn't give any information https://www.reddit.com/r/rails/comments/6cniva/rails_51_polymorphic_url_issues_during_testing/
comment
created
cbaines
username_1
I've created https://github.com/alphagov/whitehall/pull/4200 for this change now.
comment
created
cbaines
username_1
This hasn't moved forward in a while now, so I've opened https://github.com/alphagov/whitehall/pull/4236 instead.
comment