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 for "duplicate identifier" via generateIndexFile hook for multiple schemas #558

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

Conversation

simian-loco
Copy link

Instead of a global models/index.ts, when using schemas, add models/schema/index.ts index files for preventing typescript "duplicate identifier" errors when schemas contain overlapping resources (e.g. postgrest, supabase, other api-based-schema paradigms).

Copy link

sweep-ai bot commented May 15, 2024

Sweep: PR Review

Sweep is currently reviewing your pr...

@kristiandupont
Copy link
Owner

Thank you for contributing! I do not have time to look at your branch today but I will get around to it this week and merge or suggest changes!

@kristiandupont
Copy link
Owner

Sorry, that took a bit longer than I anticipated. So forgive me if I am misunderstanding, but it seems that this will always change the output in that there will always be at least one schema. If that's the case then I think this should be an option instead of the default. Or perhaps the old style should be an option, I am not sure..

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