Skip to content

Merge EF Core migrations into a single consolidated migration - #5842

Open
johnsimons wants to merge 2 commits into
masterfrom
john/merge_migrations
Open

Merge EF Core migrations into a single consolidated migration#5842
johnsimons wants to merge 2 commits into
masterfrom
john/merge_migrations

Conversation

@johnsimons

Copy link
Copy Markdown
Member

Replaces the series of incremental migrations (Initial, AddFullTextSearch, AddKnownEndpointMaxLength, AddConfiguration, AddFailedErrorImports, AddSubscriptions, DropDuplicateQueueAddress, EventLogItems, AddFailureGroupClassifierIndex, NullableEndpointAddress, AddGroupComments, AddMessageRedirects) with a single squashed migration representing the final schema state.

Replaces the series of incremental migrations (Initial, AddFullTextSearch, AddKnownEndpointMaxLength, AddConfiguration, AddFailedErrorImports, AddSubscriptions, DropDuplicateQueueAddress, EventLogItems, AddFailureGroupClassifierIndex, NullableEndpointAddress, AddGroupComments, AddMessageRedirects) with a single squashed migration representing the final schema state.

@rbev rbev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The description on this PR is misleading since it's got SQLServer changes in it too

});

migrationBuilder.CreateTable(
name: "EventLogItems",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This table name is wrong

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

How is this possible, we use dotnet ef migration

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed

@johnsimons johnsimons changed the title Merge PostgreSQL EF Core migrations into a single consolidated migration Merge EF Core migrations into a single consolidated migration Sep 2, 2026
Removes the explicit `ToTable("EventLogItems")` and `ToTable("ExternalIntegrationDispatchRequests")` calls from the shared entity configurations, allowing each provider's naming convention to take effect. PostgreSQL migrations now use snake_case table names (`event_log_items`, `external_integration_dispatch_requests`) while SQL Server retains PascalCase. Migration timestamps are updated to reflect the regeneration.
@johnsimons
johnsimons requested a review from rbev September 2, 2026 04:16
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