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

Hotfix: requested group ids arrays must be of same length #9038

Merged
merged 1 commit into from
Nov 30, 2024

Conversation

philipperolet
Copy link
Contributor

Description

Fixes issue from this kind of
monitor

The longer term fix would be to store requested ids differently:

  • either as string[] with comma-separated values;
  • or as a JSONB object

Risks

na

Deploy

front

Description
---
Fixes issue from [this kind of
monitor](https://dust4ai.slack.com/archives/C05F84CFP0E/p1732945133294399)

The longer term fix would be to store requested ids differently:
- either as string[] with comma-separated values;
- or as a JSONB object

Risks
---
na

Deploy
---
front
@philipperolet
Copy link
Contributor Author

PMRR

@philipperolet philipperolet merged commit 8c9a9b2 into main Nov 30, 2024
3 checks passed
@philipperolet philipperolet deleted the hotfix-requestedgids branch November 30, 2024 15:45
@flvndvd
Copy link
Contributor

flvndvd commented Nov 30, 2024

This is weird 🤔. Where does it happen? Conversation?

@philipperolet
Copy link
Contributor Author

yes (check the link in the pr description)

@spolu
Copy link
Contributor

spolu commented Nov 30, 2024

Thanks! Lets dig into it and find a proper solution on Monday 👍

@flvndvd
Copy link
Contributor

flvndvd commented Dec 2, 2024

Will take care of it.

@flvndvd
Copy link
Contributor

flvndvd commented Dec 2, 2024

Tracked here.

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.

3 participants