Skip to content

fix(db-postgres): port many various fixes from 3.0 #8468

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
Merged
DanRibbens merged 19 commits into from
Oct 8, 2024
Merged

fix(db-postgres): port many various fixes from 3.0 #8468

DanRibbens merged 19 commits into from
Oct 8, 2024

Conversation

@r1tsuu
Copy link
Member

@r1tsuu r1tsuu commented Sep 29, 2024 •
edited
Loading

This fixes many various issues that are already fixed in 3.0. Updates Drizzle to match beta to fix some issues #4673 #6845 #6266, prev Drizzle update PR #7460

Ported PRs:

r1tsuu and others added 12 commits September 27, 2024 23:59
@r1tsuu @DanRibbens
fix(db-postgres): localized groups/tabs and nested fields
dc38811 
Co-authored-by: Dan Ribbens <dan.ribbens@gmail.com>
@jmikrut @r1tsuu
cherry pick #7900 hasMany text, number, poly relationship, blocks, ar…
e317500 
…rays within localized fields
@r1tsuu @jmikrut
cherry pick #7962 fix(db-postgres): nested localized arrays
3a12f16 
Co-authored-by: James Mikrut <james@payloadcms.com>
@r1tsuu
fix: create new ids for blocks / arrays items before duplication
f527f74
@r1tsuu
chore: comment, remove debugger
c20088e
@r1tsuu
cherry pick #8355 fix(drizzle): array / relationship / select hasMany…
453f84c 
… inside of localized groups / tabs
@r1tsuu
cherry pick #8456 fix(drizzle): hasMany / poly relationships nested t…
7210fb9 
…o localized fields / nested blocks to localized fields (#8456)
@r1tsuu
fix(drizzle): sanitize query value uuid / number id NaN (#8369)
36bd578 
Fixes #8347 (additionally
for UUID search as well)
@r1tsuu
fix(drizzle): optimize count querying when no joins #7749
ca8ddbf
@r1tsuu
chore: remove hardcoded jest setup postgres
046f41f
@r1tsuu
fix: this.tables -> adapter.tables findMany count
d29dd15
@r1tsuu
fix: count query check for selectCountMethods instead of joinAliases
73b7da6
@r1tsuu r1tsuu mentioned this pull request Sep 30, 2024
Closed
@r1tsuu r1tsuu requested a review from denolfe as a code owner October 1, 2024 01:23
@r1tsuu r1tsuu force-pushed the fix/port-postgres-fixes branch from 58e6954 to 145590b Compare October 1, 2024 01:32
@DanRibbens DanRibbens merged commit 1347b6c into main Oct 8, 2024
@DanRibbens DanRibbens deleted the fix/port-postgres-fixes branch October 8, 2024 14:57
This was referenced Oct 14, 2024
Closed
Closed
Closed
@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2024

🚀 This is included in version v2.30.2

@r1tsuu r1tsuu mentioned this pull request Oct 18, 2024
Closed
@r1tsuu r1tsuu mentioned this pull request Oct 22, 2024
Closed
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

@denolfe denolfe Awaiting requested review from denolfe denolfe is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@r1tsuu @DanRibbens @jmikrut