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

Handle the type when an enum is used as an array in postgres #1416

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MAST1999
Copy link

When using postgres if you defined the column as an enum and made it an array, it would still show as an enum instead of an array of enums.

@MAST1999 MAST1999 changed the title Handle the generated type when an enum is used as an array in postgres Handle the type when an enum is used as an array in postgres Oct 25, 2023
@MAST1999
Copy link
Author

MAST1999 commented Oct 25, 2023

Fixes #1345

Also fixes this issue for the zod and valibot types as well.

@MAST1999 MAST1999 force-pushed the main branch 2 times, most recently from 8c9c287 to d9123da Compare October 25, 2023 03:49
@asselinpaul
Copy link

Hit this bug — appreciate the fix.

@asselinpaul
Copy link

@AndriiSherman what do we need to do to get this merged?

When using Postgres if you defined the column as an enum and made it an array, it would still show as an enum instead of an array of enums.
@itsyoboieltr
Copy link

any news on this?

@amjed-ali-k
Copy link

any updates?

@nikhilpsathyanathan
Copy link

Any workaround?

@amjed-ali-k
Copy link

amjed-ali-k commented Jan 15, 2024

Any workaround?

@nikhilpsathyanathan പെട്ടെന്ന് ആയാൽ മതിയായിരുന്നു . 💯

@renesass
Copy link

Friendly push 😊

@gregory-krieger
Copy link

Any updates ?

@ngethan
Copy link

ngethan commented May 25, 2024

Can this please be merged

@rorz
Copy link

rorz commented Jun 5, 2024

Interested in seeing this merged too!

@nikhilpsathyanathan
Copy link

Interested in seeing this merged, too!

I think it's implemented, I have seen it in the release log, check on Discord channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants