meilisearch / meilisearch-rails

Meilisearch integration for Ruby on Rails

https://www.meilisearch.com

MIT License

308 stars 47 forks source link

Modified to synchronize only when setting values have been changed #225

Closed yagince closed 1 year ago

yagince commented 1 year ago

Pull Request

Modified to synchronize only when setting values have been changed, since Meilisearch settings change is highly loaded when there is a lot of data.

Related issue

Fixes #

What does this PR do?

Fixed result to be true if the order of values is different but no change.
Fixed result to be true if the value type is Hash and the contents are the same.
Modified to synchronize only when setting values have been changed, since Meilisearch settings change is highly loaded when there is a lot of data

PR checklist

Please check if your PR fulfills the following requirements:

[ ] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[ ] Have you read the contributing guidelines?
[ ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

brunoocasali commented 1 year ago

@yagince Thanks a lot for your contribution. I'll give a look at it as soon as possible! ❤️