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

Update Knex version to fix CockroachDB relationships showing multiple times #14509

Merged
merged 2 commits into from Jul 19, 2022
Merged

Update Knex version to fix CockroachDB relationships showing multiple times #14509

merged 2 commits into from Jul 19, 2022

Conversation

christopher-kapic
Copy link

@christopher-kapic christopher-kapic commented Jul 19, 2022

Description

Knex just came out with a new release, which should fix the issue in which relational data was displayed multiple times with CockroachDB.

Fixes #

Type of Change

  • Bugfix
  • Improvement
  • New Feature
  • Refactor / codestyle updates
  • Other, please describe:

Requirements Checklist

  • New / updated tests are included (n/a)
  • All tests are passing locally
  • Performed a self-review of the submitted code

If adding a new feature:

  • Documentation was added/updated

@rijkvanzanten rijkvanzanten self-assigned this Jul 19, 2022
@rijkvanzanten rijkvanzanten added this to the Next Release milestone Jul 19, 2022
@christopher-kapic
Copy link
Author

Do I need to do anything else for this to be completed, or will you finish it from here @rijkvanzanten ?

@rijkvanzanten
Copy link
Member

I got it from here 👍🏻

@rijkvanzanten rijkvanzanten merged commit 090671e into directus:main Jul 19, 2022
@hairmare
Copy link

woohoo 🥳 thanks @christopher-kapic

qborisb pushed a commit to qdentity/directus that referenced this pull request Jul 22, 2022
… times (directus#14509)

* update knex version

* Update all knex uses

Co-authored-by: rijkvanzanten <rijkvanzanten@me.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants