migrate ignores NOT NULL → nullable on an existing column, then fails integrity check
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
Línea de trabajo
Start with the migration behavior exercised by migrations/schema.yaml and reproduce the change to public.topup.stripe_payment_intent_id against PostgreSQL. Inspect the generated migration plan and integrity check for nullability changes. Done means the normal migrate path applies the NOT NULL relaxation without --allow-destructive and the integrity check passes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Reproduction
With DataProviderMigrate 0.9.12-beta against an empty Supabase Postgres 15.1.0.117 database:
- Run
DataProviderMigrate migratewith the current NAPmainschema (migrations/schema.yaml). It createspublic.topup.stripe_payment_intent_idasText,isNullable: false. - Change only that column declaration to nullable (
type: Text, with noisNullable: false) and runmigrateagain without--allow-destructive.
The second invocation reports Migration completed successfully, then exits nonzero:
SCHEMA INTEGRITY CHECK FAILED
public.topup.stripe_payment_intent_id: nullability expected NULL but found NOT NULL
The table is empty. The plan contains no ALTER COLUMN ... DROP NOT NULL equivalent. This blocks NAP's additive Stripe Checkout migration: a PaymentIntent ID does not exist when the pending Checkout row is inserted and must become nullable before the code deploys.
Expected
Relaxing NOT NULL should be classified as additive and applied by the normal migrate path, then pass integrity verification. No raw SQL or destructive migration should be required. This is independent of the existing missing-FK-on-ALTER issue #79/#80, which also appears when the Stripe schema adds a tenant FK to an existing table.
- Lenguaje dominante
- C#
- Estrellas
- 67
- Forks
- 4
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Nimblesite/DataProvider
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Nimblesite/DataProvider#72 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Nimblesite/DataProvider#108 ·
-
A check constraint the schema cannot name can never be retired, and keeps enforcing the old ruleAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Nimblesite/DataProvider#107 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Nimblesite/DataProvider#106 ·
-
migrate --allow-destructive drops EVERY foreign key when the database already matches the schemaAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
Nimblesite/DataProvider#105 · 5 comentarios ·
Todos los issues de Nimblesite/DataProvider
Issues similares
-
Money ExploitsAbiertoS: Untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
project-wayfarer/wayfarer-14#1628 ·
Los mantenedores suelen responder en 3 días
-
:watch: Not Triaged dotnet-target-version
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
copilot documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días
-
untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
dotnet/dotnet-api-docs#13124 ·
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día