@alecslupu I was going to say that we should change the base branch from develop to the one from #8411 (tremend-cofe:upgrade/rails-6.1) but as it's from the tremend-cofe organization I think that we can't do that. So, how about if we review and merge this after we merge #8411? (So it's easier to review, if not my head can explode with that many changes at the same time 😅)

As I said in that PR, the only one that I fell is a stopper for merging was the PaperTrail warnings, and that's already fixed in #8608

@alecslupu I was going to say that we should change the base branch from develop to the one from #8411 (tremend-cofe:upgrade/rails-6.1) but as it's from the tremend-cofe organization I think that we can't do that. So, how about if we review and merge this after we merge #8411? (So it's easier to review, if not my head can explode with that many changes at the same time sweat_smile)

As I said in that PR, the only one that I fell is a stopper for merging was the PaperTrail warnings, and that's already fixed in #8608

If you manage to review and merge the #8411, then this will be a very simple one (10 - 20 files changed) to address the deprecations.

I think you may be able to see this link : i-need-another-coffee/decidim@upgrade/rails-6.1...tremend-cofe:upgrade/rails-6.1-deprecations

@alecslupu Can you rebase with develop and resolve the conflicts now that #8411 is merged? 🙏

Few things I've noticed:

ActiveModel::Errors#keys is deprecated

https://github.com/tremend-cofe/decidim/blob/c602c84ea15e721d2927fc18eb67d8676bb43dc4/decidim-elections/spec/forms/decidim/elections/admin/answer_form_spec.rb#L62

Enumerating ActiveModel::Errors as a hash has been deprecated.

https://github.com/tremend-cofe/decidim/blob/c602c84ea15e721d2927fc18eb67d8676bb43dc4/decidim-initiatives/app/commands/decidim/initiatives/admin/create_initiative_type.rb#L29

Initialization autoloaded the constant Decidim::Budgets::OrderReminderGenerator.

1. Go to the development app
2. rm -f log/development.log
3. bundle exec rails s
4. cat log/development.log

To fix this, we need to move this:
https://github.com/tremend-cofe/decidim/blob/c602c84ea15e721d2927fc18eb67d8676bb43dc4/decidim-budgets/app/services/decidim/budgets/order_reminder_generator.rb

To the lib folder and add an autoload statement for it within the module.

Rails console giving warnings with Spring

NOTE: This is fixed in another PR: #9032

As I mentioned in the previous review comments, the rails console is giving lots of warnings with spring:
#8411 (comment)

Judging by this, this is also the reason why it hangs for you (as you mentioned in that discussion):
rails/spring#396

There is quite a clean workaround for this reported at:
rails/spring-watcher-listen#15 (comment)

So, if we modify the application's config/spring.rb to apply this in Decidim, it would look something like this:

# add these to config/spring.rb

module SpringWatcherListenIgnorer
  def start
    super
    listener.ignore(/^(node_modules|storage|tmp)/)
  end
end

Spring::Watcher::Listen.prepend SpringWatcherListenIgnorer

This config file is coming from here:
https://github.com/rails/rails/blob/d38ff07a39c98f5db2b15ca3a51acee48c972205/railties/lib/rails/generators/rails/app/app_generator.rb#L123

And it is defined here:
https://github.com/rails/rails/blob/d38ff07a39c98f5db2b15ca3a51acee48c972205/railties/lib/rails/generators/rails/app/templates/config/spring.rb.tt

@alecslupu I fixed the errors with Spring in another PR: #9032

@ahukkanen I will handle the autoload in a separate PR.