-
Notifications
You must be signed in to change notification settings - Fork 2
Integrate missing features from 3.5.0 to 3.13.0 #28
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
base: 1.3.x
Are you sure you want to change the base?
Conversation
DanielBadura
commented
Dec 9, 2025
- ReadonlyStore
- StoreMigrationCommand
- Store options
- CommandBus
- QueryBus
- MessageLoader
- Multiple RetryStrategies
- Tag default ArgumentResolver for Subscribers
- Enable cryptography options
| 'event-sourcing:store:migrate', | ||
| 'migrate events from one store to another', | ||
| )] | ||
| final class StoreMigrateCommand extends Command |
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.
I think this should to the library in 4.x so that we dont need to maintain it twice or more (here and symfony-bundle)
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.
We can add this in version 3.x and use it here. We don't have to wait for version 4.x.
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.
Wups, true.
|
Need to update docs |
04956ff to
a7ca5b9
Compare
| 'event-sourcing:store:migrate', | ||
| 'migrate events from one store to another', | ||
| )] | ||
| final class StoreMigrateCommand extends Command |
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.
We can add this in version 3.x and use it here. We don't have to wait for version 4.x.
a7ca5b9 to
ebb1c0c
Compare
DanielBadura
left a comment
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.
I updated the PR again, sorry I squashed it into one commit 🙈
@DavidBadura
ebb1c0c to
b002280
Compare
* ReadonlyStore * StoreMigrationCommand * Store options * CommandBus * QueryBus * MessageLoader * Multiple RetryStrategies * Tag default ArgumentResolver for Subscribers * Enable cryptography options
b002280 to
64422f9
Compare