Skip to content

Prevent ransack to be updated to vs 2.3 due a bug with polymorphic joins#5303

Merged
oriolgual merged 3 commits intodecidim:masterfrom
Platoniq:fix-ransack-version
Aug 23, 2019
Merged

Prevent ransack to be updated to vs 2.3 due a bug with polymorphic joins#5303
oriolgual merged 3 commits intodecidim:masterfrom
Platoniq:fix-ransack-version

Conversation

@microstudi
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Proposals search is broken if bundler upgrades the ransack gem to version 2.3 due (probably) to this bug:
activerecord-hackery/ransack#1039

Until resolved we should ensure that ransack stays at 2.1.x versions.

The error thrown is:

undefined method `polymorphic?' for ActiveRecord::Reflection::PolymorphicReflection

Caused in decidim-core/lib/decidim/amendable.rb when scopes only_amendables or only_emendations are used.

📌 Related Issues

  • Related to #?
  • Fixes #?

📋 Subtasks

  • Add CHANGELOG entry
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

📷 Screenshots (optional)

Description

@microstudi microstudi requested a review from a team as a code owner August 21, 2019 11:48
Copy link
Copy Markdown

@qlty-cloud-legacy qlty-cloud-legacy bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 6603 lines exceeds the maximum allowed for the inline comments feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants