-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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: v12 migrate error - unknown column ‘mandatory_depends_on’ #27897
Conversation
@Mergifyio backport version-12-hotfix version-13-hotfix |
Command
|
Codecov Report
@@ Coverage Diff @@
## develop #27897 +/- ##
===========================================
+ Coverage 54.39% 54.41% +0.01%
===========================================
Files 1254 1254
Lines 67717 67717
===========================================
+ Hits 36838 36845 +7
+ Misses 30879 30872 -7
|
Head branch was pushed to by a user without write access
@ankush Linters failing due to another piece of code. |
Have fixed this. Can merge now :) |
* fix: v12 doesn't have mandatory_depends_on field * fix: move update_vehicle_no_reqd_condition to v13 * fix: move update_vehicle_no_reqd_condition to v13 * fix: file name missing .py * refactor!: add back empty line * fix: linters issue (cherry picked from commit 7acdcc7) # Conflicts: # erpnext/accounts/report/consolidated_financial_statement/consolidated_financial_statement.py # erpnext/patches.txt # erpnext/patches/v13_0/update_vehicle_no_reqd_condition.py
* fix: v12 doesn't have mandatory_depends_on field * fix: move update_vehicle_no_reqd_condition to v13 * fix: move update_vehicle_no_reqd_condition to v13 * fix: file name missing .py * refactor!: add back empty line * fix: linters issue (cherry picked from commit 7acdcc7) # Conflicts: # erpnext/patches.txt
Command
|
…ort #27897) (#27900) * fix: v12 migrate error - unknown column ‘mandatory_depends_on’ (#27897) * fix: v12 doesn't have mandatory_depends_on field * fix: move update_vehicle_no_reqd_condition to v13 * fix: move update_vehicle_no_reqd_condition to v13 * fix: file name missing .py * refactor!: add back empty line * fix: linters issue (cherry picked from commit 7acdcc7) # Conflicts: # erpnext/patches.txt * fix: resolve conflicts Co-authored-by: Dany Robert <rtdany10@gmail.com> Co-authored-by: Ankush Menat <ankush@iwebnotes.com>
…e#27897) * fix: v12 doesn't have mandatory_depends_on field * fix: move update_vehicle_no_reqd_condition to v13 * fix: move update_vehicle_no_reqd_condition to v13 * fix: file name missing .py * refactor!: add back empty line * fix: linters issue
v12 doesn't have mandatory_depends_on field.
Removed a patch that referenced to the said field on v12.
Closes frappe/frappe#14335 (comment)