Skip to content

Conversation

vincent-pochet
Copy link
Collaborator

Context

This PR fixes the migration to add a not null constraint on the payment_provider_customers table.
The migration task was missing an unscoped scope, to take the deleted resources into account in the migration.

Description

To make sure it will not cause any issue when running the migration, a new DB migration is added before the one changing the constraint.

@vincent-pochet vincent-pochet changed the title fix: Migrate deleted payment_provider_customers fix: Ensure not null organization_id on deleted payment_provider_customers Jul 7, 2025
Copy link
Contributor

@julienbourdeau julienbourdeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh no 😟

@vincent-pochet vincent-pochet merged commit f712f1e into main Jul 7, 2025
14 checks passed
@vincent-pochet vincent-pochet deleted the fix-migration branch July 7, 2025 13:59
diegocharles pushed a commit that referenced this pull request Jul 11, 2025
…omers (#3935)

## Context

This PR fixes the migration to add a not null constraint on the
`payment_provider_customers` table.
The migration task was missing an `unscoped` scope, to take the deleted
resources into account in the migration.

## Description

To make sure it will not cause any issue when running the migration, a
new DB migration is added before the one changing the constraint.
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