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.

Sign up for GitHub

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

Jump to bottom

fix(ui): scheduled publish not showing related events in postgres #10481

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(ui): scheduled publish not showing related events in postgres #10481

wants to merge 1 commit into from

Conversation

paulpopus
Copy link
Contributor

@paulpopus paulpopus commented Jan 9, 2025 •
edited
Loading

Since postgres uses number IDs by default, when we were storing the relationship field value with postgres we weren't able to query it

This fixes that problem by casting the ID to always a string making it safe for querying inside the JSON field

@paulpopus paulpopus requested a review from DanRibbens January 9, 2025 21:02
@DanRibbens
Copy link
Contributor

@paulpopus please add a description before this is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DanRibbens DanRibbens DanRibbens approved these changes

Assignees
No one assigned
Labels
created-by: Payload team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@paulpopus @DanRibbens