Skip to content
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(dp): Fix migration #12255

Merged
merged 1 commit into from Mar 24, 2022
Merged

fix(dp): Fix migration #12255

merged 1 commit into from Mar 24, 2022

Conversation

jkmar
Copy link
Contributor

@jkmar jkmar commented Mar 23, 2022

Summary

In latest migration json default was incorrectly formatted,
And although migration was generated it couldn't have been applied.

Signed-off-by: Kuba Marciniszyn kuba@freedomfi.com

@jkmar jkmar requested review from xbend, WojSad and a team March 23, 2022 15:46
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines. label Mar 23, 2022
In latest migration json default was incorrectly formatted,
And although migration was generated it couldn't have been applied.

Signed-off-by: Kuba Marciniszyn <kuba@freedomfi.com>
@jkmar jkmar merged commit 2a51148 into magma:master Mar 24, 2022
@jkmar jkmar deleted the fix_migration branch March 24, 2022 11:21
@github-actions
Copy link
Contributor

github-actions bot commented Mar 24, 2022

Unit Test Results

157 files  157 suites   2h 5m 4s ⏱️
307 tests 304 ✔️ 3 💤 0

Results for commit 2a51148.

♻️ This comment has been updated with latest results.

ardzoht pushed a commit that referenced this pull request Mar 30, 2022
In latest migration json default was incorrectly formatted,
And although migration was generated it couldn't have been applied.

Signed-off-by: Kuba Marciniszyn <kuba@freedomfi.com>
emakeev pushed a commit to emakeev/magma that referenced this pull request Aug 5, 2022
In latest migration json default was incorrectly formatted,
And although migration was generated it couldn't have been applied.

Signed-off-by: Kuba Marciniszyn <kuba@freedomfi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants