-
Notifications
You must be signed in to change notification settings - Fork 25
refactor!: migration logging system #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
Draft
larskemper
wants to merge
42
commits into
trunk
Choose a base branch
from
feature/migration-logging-refactor
base: trunk
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
42 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 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
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 was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
todo before merge: check all commit message for conventional commit and don't squash merge