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

Drop postgreSQL column default if functional values #38

Merged
merged 8 commits into from
Jul 22, 2024

Conversation

PineappleIOnic
Copy link
Member

PostgreSQL's column_default can potentially be a SQL/PostgreSQL statement for auto increments, this is unsupported in Appwrite and causes issues when migrating these.

This PR also detaches warnings from Resources allowing for easier emitting and tracking of warnings, similar to what we did with errors.

@PineappleIOnic PineappleIOnic changed the title Drop postgreSQL column default if their not a normal value Drop postgreSQL column default if functional values Jul 22, 2024
@PineappleIOnic PineappleIOnic requested a review from abnegate July 22, 2024 04:03
src/Migration/Target.php Outdated Show resolved Hide resolved
src/Migration/Target.php Outdated Show resolved Hide resolved
PineappleIOnic and others added 2 commits July 22, 2024 17:06
@abnegate abnegate merged commit f18d44d into main Jul 22, 2024
2 checks passed
@abnegate abnegate deleted the feat-fix-postgres-attribute-migration branch July 22, 2024 09:27
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.

2 participants