db:migrate fails on AddIndexToEventsOnPosAndParentId

I try to skip this migrate like this :

docker exec -ti loomio-db /bin/bash
su postgres
psql loomio_production
insert into schema_migrations (version) values ('20170826033216');

And run again

docker-compose run app rake db:migrate


4 others migrates fail for various reasons. I skipped them the same way and it seems to work.

But I don’t know if it’s dangerous for user’s data. I don't really understand the implications of these changes.
For now, I've only done this in a testing environment.
Do you think I can apply this in production without too much damage?

no. all migrations are required. skipping them will have bad consequences :)

@josephk - it looks like position is being interpreted as a keyword rather than a column name. I suggest you try editing the migration to be:

add_index(:events, [:parent_id, :position], {:order=>{position: :desc} , :where=>"(parent_id IS NOT NULL)"})


and see if that helps

Ok, thank you.
It works for the first migrate error but I still get this error with the second one :
https://github.com/loomio/loomio/blob/master/db/migrate/20171221225240_migrate_versions_to_eventables_attempt_2.rb
```
Migrating to MigrateVersionsToEventablesAttempt2 (20171221225240)

== 20171221225240 MigrateVersionsToEventablesAttempt2: migrating ==============

rake aborted!

StandardError: An error has occurred, this and all later migrations canceled:

undefined method token' for #<FormalGroup:0x000055db93b3e290>
Did you mean? to_key
``

And the third one (when the second is skipped) :
https://github.com/loomio/loomio/blob/master/db/migrate/20180305031829_add_notification_model.rb
```
Migrating to AddNotificationModel (20180305031829)

== 20180305031829 AddNotificationModel: migrating =============================

rake aborted!

StandardError: An error has occurred, this and all later migrations canceled:

undefined method discussion' for nil:NilClass
``
On the other hand, it solved the problem on errors 4 and 5.

comment out initialized_with_token :token (line 10) in /app/models/group.rb for the migration (then uncomment it once you're through).

AddNotificationModel just updates precomputed strings.. it's technically optional.. so if it still fails you can skip that one, however I don't think you should be skipping any other migrations... that would give undefined behaviour..

good luck!

It works! Thank you very much!

That's great! Pleased to hear it.

It would be good to talk more about how you can contribute back to our open source resources.

I'd be particularly keen to talk about french translation of our help.loomio.org resource if you have anyone who could help with that.