feat(smart-field): add new options to support native polymorphic fields

[nicolasalexandre9]

field :addressable_type, type: 'String', polymorphic_key: true, is_filterable: true do
  object.addressable_type
end

field :addressable_id, type: 'String', polymorphic_key: true do
  object.addressable_type
end

Definition of Done

General

• Write an explicit title for the Pull Request, following Conventional Commits specification
• Test manually the implemented changes
• Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

• Consider the security impact of the changes made

[codeclimate]

Code Climate has analyzed commit 4da464a and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (55% is the threshold).

This pull request will bring the total coverage in the repository to 51.0%.

View more on Code Climate.

[forest-bot]

🎉 This PR is included in version 9.5.0 🎉

The release is available on:

• GitHub release
• v9.5.0

Your semantic-release bot 📦🚀

[lostie]

I have tested these changes, but when I define the field for the polymorphic column with polymorphic_key, it still require us to provide the options like is_filterable, is_sortable in order to enable them.

A nice to have would be if those options were by default true when we mark the field with polymorphic_key: true, since those columns are usually filterable and sortable by default anyway.

[nicolasalexandre9]

Hi @lostie, thanks for your feedback.

It sounds like a great idea, so we will be opening a PR soon to set these options to true by default in this case.
I'll let you know when the PR is open.