Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove n+1 talks controller with Meilisearch #56

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

adrienpoly
Copy link
Owner

No description provided.

@useattractor
Copy link

useattractor bot commented Sep 14, 2023

remove n+1 talks controller with Meilisearch 🔗

Stats

Language Score Trend
Ruby 20.33 (from 20.22) 📈 0.58%
JavaScript 39.69 (from 39.69) 📉 0.0%

Trends

Most Improved Largest Declines
Ruby No decreases for Ruby detected app/models/talk.rb and app/controllers/talks_controller.rb
JavaScript No decreases for JavaScript detected No increases for JavaScript detected

To-Dos

New to Refactor Refactored
Ruby No new To-Dos for Ruby detected No completed To-Dos for Ruby detected
JavaScript No new To-Dos for JavaScript detected No completed To-Dos for JavaScript detected

@adrienpoly adrienpoly merged commit 41de772 into main Sep 14, 2023
3 checks passed
@adrienpoly adrienpoly deleted the remove-n+1-talks-controller-with-search branch September 14, 2023 06:16
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.

1 participant