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

fix(tags-db): correct tag and study_tag migration script #1934

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

laurent-laporte-pro
Copy link
Contributor

@laurent-laporte-pro laurent-laporte-pro commented Feb 14, 2024

Avoid bulk insertion if the list of values to insert is empty.

fix #1929

@laurent-laporte-pro laurent-laporte-pro added bug Something isn't working hot back-end labels Feb 14, 2024
@laurent-laporte-pro laurent-laporte-pro added this to the v2.16.5 milestone Feb 14, 2024
@laurent-laporte-pro laurent-laporte-pro self-assigned this Feb 14, 2024
@laurent-laporte-pro laurent-laporte-pro changed the title bugfix(tags-db): correct tag and study_tag migration script fix(tags-db): correct tag and study_tag migration script Feb 14, 2024
@laurent-laporte-pro laurent-laporte-pro force-pushed the bugfix/correct-tag-and-study_tag-migration branch from ec78792 to 2d7f174 Compare February 14, 2024 22:33
Copy link
Contributor

@mabw-rte mabw-rte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faut utiliser une liste.

Avoid bulk insertion if the list of values to insert is empty.
@laurent-laporte-pro laurent-laporte-pro force-pushed the bugfix/correct-tag-and-study_tag-migration branch from 2d7f174 to 2cfa552 Compare February 15, 2024 09:50
@laurent-laporte-pro laurent-laporte-pro merged commit 8bdc837 into dev Feb 15, 2024
7 checks passed
@laurent-laporte-pro laurent-laporte-pro deleted the bugfix/correct-tag-and-study_tag-migration branch February 15, 2024 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end bug Something isn't working hot size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants