Skip to content

Conversation

VincentMolinie
Copy link
Member

The query was badly generated by sequelize because we were trying to actually "hack" how sequelize work. Instead of doing that I just generated the query manually

Pull Request checklist:

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Create automatic tests
  • Test manually the implemented changes
  • Review my own code (indentation, syntax, style, simplicity, readability)
  • Wonder if you can improve the existing code

@arnaudbesnier
Copy link
Contributor

@larcin larcin assigned VincentMolinie and unassigned larcin Oct 1, 2020
@VincentMolinie VincentMolinie merged commit 0e90750 into master Oct 1, 2020
@VincentMolinie VincentMolinie deleted the fix/leaderboard-chart-many-to-many branch October 1, 2020 09:30
forest-bot added a commit that referenced this pull request Oct 1, 2020
## [6.3.12](v6.3.11...v6.3.12) (2020-10-01)

### Bug Fixes

* **leaderboard chart:** fix query on many to many relationship ([#510](#510)) ([0e90750](0e90750))
@forest-bot
Copy link
Member

🎉 This PR is included in version 6.3.12 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants