Skip to content

Chore: 13247/change table swag migration logging #80

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.

Sign up for GitHub

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

Jump to bottom
Merged
DennisGarding merged 24 commits into from
Nov 7, 2025
Merged

Chore: 13247/change table swag migration logging #80

DennisGarding merged 24 commits into from
Nov 7, 2025

Conversation

@DennisGarding
Copy link
Contributor

@DennisGarding DennisGarding commented Oct 31, 2025 •
edited
Loading

Closes: #80

  • add relation to "entity_id"
  • add index to "entity_id"
  • remove field "used_mapping"

@DennisGarding
Merge branch 'trunk' of github.com:shopware/SwagMigrationAssistant in…
d6f73ba 
…to feature/migration-logging-refactor
@DennisGarding
Merge branch 'feature/migration-logging-refactor' of github.com:shopw…
3cf6d32 
…are/SwagMigrationAssistant into feature/migration-logging-refactor
@DennisGarding
Merge branch 'feature/migration-logging-refactor' of github.com:shopw…
5ad6e0a 
…are/SwagMigrationAssistant into feature/migration-logging-refactor
@DennisGarding
Merge branch 'feature/migration-logging-refactor' of github.com:shopw…
0bc9d86 
…are/SwagMigrationAssistant into feature/migration-logging-refactor
@DennisGarding
Merge branch 'feature/migration-logging-refactor' of github.com:shopw…
be1132a 
…are/SwagMigrationAssistant into feature/migration-logging-refactor
@DennisGarding
Merge branch 'feature/migration-logging-refactor' of github.com:shopw…
fe40fa5 
…are/SwagMigrationAssistant into feature/migration-logging-refactor
@DennisGarding
chore: Add and remove column to swag_migration_logging table
667907a
@DennisGarding
Fix cs
48dedae
@DennisGarding
Fix issues
2104a2a
@DennisGarding DennisGarding self-assigned this Nov 3, 2025
MalteJanz
MalteJanz reviewed Nov 3, 2025
View reviewed changes
larskemper
larskemper approved these changes Nov 3, 2025
View reviewed changes
MalteJanz
MalteJanz requested changes Nov 3, 2025
View reviewed changes
Copy link
Contributor

@MalteJanz MalteJanz left a comment

Choose a reason for hiding this comment

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

Are these already all places where logs are created? I guess any logs that are created in converters are being covered by your other PR then? #79

MalteJanz
MalteJanz reviewed Nov 4, 2025
View reviewed changes
MalteJanz
MalteJanz reviewed Nov 4, 2025
View reviewed changes
MalteJanz
MalteJanz reviewed Nov 4, 2025
View reviewed changes
MalteJanz
MalteJanz reviewed Nov 4, 2025
View reviewed changes
MalteJanz
MalteJanz reviewed Nov 4, 2025
View reviewed changes
MalteJanz
MalteJanz reviewed Nov 4, 2025
View reviewed changes
MalteJanz
MalteJanz approved these changes Nov 6, 2025
View reviewed changes
@DennisGarding DennisGarding changed the title 13247/change table swag migration logging Chore: 13247/change table swag migration logging Nov 6, 2025
@DennisGarding DennisGarding merged commit 8a16844 into feature/migration-logging-refactor Nov 7, 2025
7 checks passed
@DennisGarding DennisGarding deleted the 13247/change-table-swagMigrationLogging branch November 7, 2025 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larskemper larskemper larskemper approved these changes

@MalteJanz MalteJanz MalteJanz approved these changes

@jozsefdamokos jozsefdamokos Awaiting requested review from jozsefdamokos

@vintagesucks vintagesucks Awaiting requested review from vintagesucks

Assignees

@DennisGarding DennisGarding

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@DennisGarding @MalteJanz @larskemper