rails 6.1: s/arel_attribute/arel_table/

ManageIQ / inventory_refresh

Apache License 2.0

1 stars 23 forks source link

rails 6.1: s/arel_attribute/arel_table/ #98

Closed kbrock closed 2 years ago

kbrock commented 2 years ago

arel_attribute has gone away Since this is referring to the primary key, this works for us in all versions of rails

So no need to turn key merge this with manageiq core.

kbrock commented 2 years ago

@agrare if manageiq uses 2.2, do we need this patch also on for 2.2? Do I need to create a pr/patch for 2.2?

agrare commented 2 years ago

@kbrock I already cherry-picked this back to the v0.2.z branch. I assume we'll have a number of rails 6.1 changes so I'm putting off a release until then