-
Notifications
You must be signed in to change notification settings - Fork 62
fix: no migrations on worker and provider startup #1410
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
fix: no migrations on worker and provider startup #1410
Conversation
0042b53
to
2b9b696
Compare
|
also correctly set 'poolSize' when initializing Typeorm DataSource and add some unit tests for TypeormService
2b9b696
to
d5faa15
Compare
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.
looks good, do you want me to test it or shall we do it after merge?
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.
Great work @mkazlauskas 🚀
Context
See commit log
LW-10831
Proposed Solution
Important Changes Introduced