-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
Upgrade Mongock dependency to v5.1.0 #279
Comments
@mraible, Since you already worked on this part before, do you want to take it up? |
I'm can take a look on Monday. Steps to reproduce would be awesome. |
@mraible, Sorry for the late reply. Lately I was bit occupied and could not spend much time over here. This ticket is to upgrade the dependencies and adjust code. In the linked commit, I see you had already tried/explored most of the changes. Let me know if you want to continue to take it to completion. |
I started on it |
I was able to make some progress and resolve most of the issues. However, native tests do yet work and fail due to a similar error as in the previous version. If we are fine with this limitation, then, I can try templatizing these changes and raise PR in the next few days. |
Sounds good! |
Adding a bug bounty. |
Is your feature request related to a problem? Please describe.
v5.1.0
contains some breaking changes. Refer migration guideDescribe the solution you'd like
Should support the latest stable release.
Describe alternatives you've considered
N/A
Additional context
Refer 3cf3bd5
The text was updated successfully, but these errors were encountered: