-
Notifications
You must be signed in to change notification settings - Fork 25
feat: error resolution #41
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
Merged
Malte Janz (MalteJanz)
merged 83 commits into
trunk
from
feature/migration-logging-refactor
Feb 27, 2026
Merged
Changes from all commits
Commits
Show all changes
83 commits
Select commit
Hold shift + click to select a range
e0d0cd5
feat: add migration logging required fields (#38)
larskemper d2468fd
refactor: add migration logging optional fields (#40)
larskemper 638b84d
refactor!: migration connection usage (#43)
larskemper d077e38
fix: phpstan log array types (#47)
larskemper 5d495b8
feat: fill logs with meaningful data src/Migration (#46)
jozsefdamokos 4a5b659
feat: add iterative log building helper (#48)
larskemper b50abbf
feat: fill logs with meaningful data (#49)
larskemper cb0f6b1
docs: update changelog
larskemper 8646a89
feat: refactor connection credentials page (#42)
jozsefdamokos 62cb5dd
refactor!: convert to ts (#39)
larskemper 54d4550
fix: phpstan for feature branch (#52)
larskemper 8536708
feat: add migration fixes table (#56)
larskemper f69773c
feat: add migration validation of converted data (#51)
larskemper d5d1cdc
refactor: cleanup wording (#59)
larskemper 19f4267
chore: drop migration cli commands (#60)
larskemper b93a071
fix: adjust profile icon paths (#61)
vintagesucks 976d4b4
refactor: use meteor tokens (#58)
larskemper fc4372d
fix: adjust log file generation (#64)
larskemper 4ed6c64
refactor: improve ui/ux (#68)
larskemper 001ed93
12367/add entity classes (#76)
DennisGarding 5be00ca
feat: checksum and reset via mq (#57)
larskemper 6deb24e
Merge remote-tracking branch 'origin/trunk' into feature/migration-lo…
larskemper 85bccaa
fix: phpstan
larskemper 271b787
fix: phpunit
larskemper 35709d7
feat: apply fixes prototype (#63)
DennisGarding 63dbef2
feat: split message queue process (#74)
jozsefdamokos 7ce1a74
feat: add acl constraints (#77)
larskemper d6af580
13248/change swag migration fix table (#81)
DennisGarding 8a16844
Chore: 13247/change table swag migration logging (#80)
DennisGarding ac67406
refactor: change swag migration mapping (#82)
DennisGarding 222c0bd
refactor: remove main mapping id from migration fix (#83)
larskemper 3356c13
ci: add check for conventional commits (#84)
larskemper 6c36b6d
refactor: re-group migrations (#86)
larskemper 7f5da1d
refactor: load fixes (#88)
DennisGarding 4409572
refactor: use xxh hash (#85)
larskemper a05f7e7
feat: add error resolution ui (#53)
larskemper 8f780ff
chore: check naming consistency across error resolution (#94)
larskemper de33168
chore: fix package de sync (#95)
larskemper 5d4ec50
refactor: remove validation from converter (#79)
DennisGarding 134e9eb
test: error resolution ui (#90)
larskemper 14f548b
fix: change connection modal (#98)
vintagesucks aba7961
fix: typos in error resolution "no option available" banner (#101)
vintagesucks 0b0a124
refactor: unify public api (#100)
larskemper 1cea903
refactor: use domain exception (#102)
larskemper a288248
feat: prevent multiple connections to the same system (#93)
larskemper 590c900
feat: prepare ui for different resolving types (#108)
larskemper 538ae17
fix: abort mq pickup (#106)
larskemper 7da8e5c
docs: changelog and upgrade (#103)
larskemper 88c2a58
feat: add reset of single fixes (#109)
larskemper 3999eb6
fix: currency conversion (#111)
jozsefdamokos 8cf9795
fix: use item-specific tax in order conversion (#114)
vintagesucks 4fc04a4
Merge branch 'trunk' of github.com:shopware/SwagMigrationAssistant in…
MalteJanz 856c558
fix: try to fix phpunit tests
MalteJanz abde712
prettier
vintagesucks c29b1e0
fix: DataProviderControllerTest
jozsefdamokos 9e168a1
Merge branch 'trunk' into feature/migration-logging-refactor
vintagesucks 2f9d9cf
refactor!: remove "Delete migration data" history action (#115)
vintagesucks adbd55e
test: add visual tests (#123)
larskemper b8f5f04
fix: missing migration state checks (#121)
larskemper 144fa7d
refactor!: refactored log classes naming pattern (#124)
larskemper d6b9251
fix: phpstan (#129)
larskemper a8d139f
refactor: logging service (#120)
larskemper d1b70fd
fix: use stable ID for `main_category` mapping (#125)
vintagesucks c193560
fix: default category type (#127)
jozsefdamokos 2a9950f
refactor: select all performance (#107)
ennasus4sun 9cab510
fix: log naming pattern (#130)
larskemper aa9189f
refactor: migration validation (#117)
larskemper d7a2bcb
fix: selected counter in select all mode (#131)
vintagesucks 404abac
test: migration ats (#126)
larskemper 45d1f46
fix: revisit ui (#122)
larskemper 9863dc1
fix: phpstan unserialize errors (#134)
MalteJanz 103a6d1
chore: benchmark and performance improvements (#132)
MalteJanz e719441
chore: align cs fixer rules (#135)
larskemper 156846b
fix: restore converter checks (#133)
larskemper cbdfa30
feat: enable required field validation for nested entities (#136)
larskemper d462ba5
fix: display nested resolution (#139)
larskemper 2aba997
fix: select all item count (#138)
larskemper f3b76bb
fix: skip app-owned custom field sets (#141)
vintagesucks 270e9b5
fix: validate new connection before storing (#137)
ennasus4sun bdff587
chore: prepare release 16.0.0 (#140)
larskemper 77b5249
Merge branch 'trunk' into feature/migration-logging-refactor
MalteJanz d97a09f
fix: strip exception traces from snapshot test (#142)
vintagesucks c26f0ad
fix: acceptance test expected entity counts (#143)
MalteJanz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,99 @@ | ||
| name: Acceptance | ||
|
|
||
| on: | ||
| schedule: | ||
| - cron: '0 3 * * *' # nightly at 3 am utc | ||
| workflow_dispatch: | ||
| inputs: | ||
| update_snapshots: | ||
| description: 'Update snapshots' | ||
| required: false | ||
| type: boolean | ||
| default: false | ||
| push: | ||
| branches: | ||
| - trunk | ||
| pull_request: | ||
| paths: | ||
| - src/** | ||
| - tests/acceptance/** | ||
| - .github/workflows/acceptance.yaml | ||
|
|
||
| jobs: | ||
| playwright: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| PLATFORM_BRANCH: ["trunk"] | ||
| services: | ||
| mysql: | ||
| image: mysql:8.0 | ||
| env: | ||
| MYSQL_ROOT_PASSWORD: root | ||
| MYSQL_DATABASE: shopware | ||
| ports: | ||
| - 3306:3306 | ||
| env: | ||
| DATABASE_URL: mysql://root:root@127.0.0.1:3306/shopware | ||
|
|
||
| steps: | ||
| - uses: shopware/github-actions/setup-extension@main | ||
| with: | ||
| extensionName: ${{ github.event.repository.name }} | ||
| shopwareVersion: ${{ matrix.PLATFORM_BRANCH }} | ||
| install: true | ||
| install-admin: true | ||
| install-storefront: false | ||
| mysqlVersion: skip | ||
| env: prod | ||
| extraRepositories: | | ||
| { | ||
| "${{ github.event.repository.name }}": { | ||
| "type": "path", | ||
| "url": "custom/plugins/${{ github.event.repository.name }}", | ||
| "symlink": true | ||
| } | ||
| } | ||
|
|
||
| - name: Setup Migration Assistant | ||
| run: | | ||
| composer run build:js:admin | ||
| cd custom/plugins/${{ github.event.repository.name }}/tests | ||
| mysql -uroot < _fixtures/database/shopware55corrupted.sql | ||
|
|
||
| - name: Cache Playwright browsers | ||
| uses: actions/cache@v4 | ||
| with: | ||
| path: ~/.cache/ms-playwright | ||
| key: playwright-${{ runner.os }}-${{ hashFiles('custom/plugins/**/tests/acceptance/package-lock.json') }} | ||
|
|
||
| - name: Install playwright | ||
| working-directory: custom/plugins/${{ github.event.repository.name }}/tests/acceptance | ||
| run: | | ||
| npm ci | ||
| npx playwright install --with-deps | ||
|
|
||
| - name: Run Playwright | ||
| working-directory: custom/plugins/${{ github.event.repository.name }}/tests/acceptance | ||
| env: | ||
| DATABASE_URL: mysql://root:root@127.0.0.1:3306/shopware55 | ||
| UPDATE_SNAPSHOTS: ${{ github.event.inputs.update_snapshots }} | ||
| run: npm run test | ||
|
|
||
| - uses: actions/upload-artifact@v4 | ||
| if: always() | ||
| with: | ||
| name: e2e-${{ github.event.repository.name }}-${{ github.sha }}-${{ matrix.PLATFORM_BRANCH }} | ||
| path: | | ||
| custom/plugins/${{ github.event.repository.name }}/tests/acceptance/test-results/ | ||
| custom/plugins/${{ github.event.repository.name }}/tests/acceptance/playwright-report/ | ||
| retention-days: 3 | ||
|
|
||
| - name: Upload updated snapshots | ||
| if: github.event.inputs.update_snapshots == 'true' | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: visual-snapshots-${{ matrix.PLATFORM_BRANCH }} | ||
| path: custom/plugins/${{ github.event.repository.name }}/tests/acceptance/snapshots/ | ||
| retention-days: 3 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| name: Check PR title | ||
|
|
||
| on: | ||
| pull_request: | ||
| types: | ||
| - opened | ||
| - reopened | ||
| - edited | ||
| - synchronize | ||
|
|
||
| permissions: | ||
| pull-requests: read | ||
|
|
||
| jobs: | ||
| lint: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: amannn/action-semantic-pull-request@48f256284bd46cdaab1048c3721360e808335d50 # 6.1.1 | ||
| with: | ||
| types: | | ||
| feat | ||
| fix | ||
| chore | ||
| docs | ||
| style | ||
| refactor | ||
| perf | ||
| test | ||
| build | ||
| ci | ||
| revert | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,32 @@ | ||
| .idea | ||
| .vscode | ||
| coverage/ | ||
| .phpunit.cache | ||
| .phpunit.result.cache | ||
| vendor | ||
| node_modules | ||
| phpstan.neon | ||
| # ide | ||
| .idea/ | ||
| .vscode/ | ||
|
|
||
| # os | ||
| .DS_Store | ||
|
|
||
| # node | ||
| node_modules/ | ||
| *.hot-update.js | ||
| /var/ | ||
|
|
||
| # php | ||
| vendor/ | ||
| phpstan.neon | ||
| composer.lock | ||
| src/Resources/public/administration/ | ||
| .phpunit.cache | ||
| .phpunit.result.cache | ||
|
|
||
| # tests | ||
| coverage/ | ||
|
|
||
| # build | ||
| src/Resources/public/static/ | ||
| src/Resources/public/administration/ | ||
| src/Resources/app/administration/.tmp | ||
|
|
||
| # runtime | ||
| /var/ | ||
| .env | ||
|
|
||
| # plugin | ||
| /migration_assistant.cache |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| **/node_modules | ||
| **/static | ||
| **/.tmp | ||
| **/public/ | ||
| **/coverage/ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| { | ||
| "singleQuote": true, | ||
| "tabWidth": 4, | ||
| "printWidth": 125, | ||
| "trailingComma": "all", | ||
| "multilineArraysWrapThreshold": 1, | ||
| "plugins": [ | ||
| "prettier-plugin-multiline-arrays" | ||
| ] | ||
| } |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.