-
-
Notifications
You must be signed in to change notification settings - Fork 729
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
[16.0][MIG] stock_demand_estimate_matrix: Migration to 16.0 #1796
[16.0][MIG] stock_demand_estimate_matrix: Migration to 16.0 #1796
Conversation
no need to modify action, once the tree view is set as editable it will work as itended, no need to remove form from the view modes.
c1a5ac0
to
ca7c6fb
Compare
/ocabot migration stock_demand_estimate_matrix |
ca7c6fb
to
0437cc8
Compare
@kaynnan Can you add the following commit? ForgeFlow@f890a86 Improves the user interface. Otherwise the matrix and product list are not seen in full width in the wizard. |
@JordiBForgeFlow Thanks for the improvement. I just updated the pull request, including your commit, and i squashed it with my commit "MIG" to keep the history more organized and clean. |
de3a3b3
to
a752179
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.
Functional review, LGTM
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 112eb68. Thanks a lot for contributing to OCA. ❤️ |
cc @marcelsavegnago
#1494