search

decidim / metadecidim

This is the repository for the code of Metadecidim community, we experiment with the new features, we are an experimental community
https://meta.decidim.org
8 stars 7 forks source link

update decidim before omniauth tenant changes #56

Closed microstudi closed 4 years ago

microstudi commented 4 years ago

Currently metadecidim is suffering from this bug:

https://github.com/decidim/decidim/pull/5560

trying to update it to the latest master bring a new issue related to this PR: https://github.com/decidim/decidim/commit/012b5b1342824f5d5566d9aad0fb55705e8be8f8

The bug is that server does not start because of a duplicated omniauth route. This is related to the fact that metadecidim has the initializer config/initializers/omniauth_decidim.rb configured and the tenant PR has some sort of incompatibility.

Until that is fixed I propose to upgrade metadecidim to solve the more urgent bug (the first).

tramuntanal commented 4 years ago

Hi @microstudi ,

I've found that the culprit is a double declaration of the same OAuth provider. See commit's description https://github.com/decidim/metadecidim/pull/56/commits/5038e784d2c1d4ba269278f5362c089668f530e6

With this change it works locally.

microstudi commented 4 years ago

So great!