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] base_import_module: activate from form view and upgrade #157008

Conversation

arpa-odoo
Copy link
Contributor

@arpa-odoo arpa-odoo commented Mar 8, 2024

Before this commit

Unable to activate the industry from the form view in the industry section. Additionally, the button in the form for upgrading would be called the button_immediate_upgrade method, resulting in errors.

After this commit

Activation of the industry from the form view in the industry section is now possible. Furthermore, the issue with upgrading the industry from the form view has been resolved.

task-3775729

@robodoo
Copy link
Contributor

robodoo commented Mar 8, 2024

@C3POdoo C3POdoo added the RD research & development, internal work label Mar 8, 2024
Copy link
Contributor

@vava-odoo vava-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @arpa-odoo
Thanks for this! I validate the part on the form view, but I don't get the "upgrade" part.
Also, while you're at it, could you also fix the tree view?
Cheers!

addons/base_import_module/views/ir_module_views.xml Outdated Show resolved Hide resolved
@arpa-odoo arpa-odoo force-pushed the 17.0-base_import_module-fix-activate-arpa branch from d630e49 to 6557088 Compare March 18, 2024 09:49
@arpa-odoo arpa-odoo force-pushed the 17.0-base_import_module-fix-activate-arpa branch from 6557088 to b3646a3 Compare March 27, 2024 08:50
Before this commit
==================
Unable to activate the industry from the form view in the industry section.
Additionally, the button in the form for upgrading would be called the
button_immediate_upgrade method, resulting in errors.

After this commit
=================
Activation of the industry from the form view in the industry section is now
possible. Furthermore, the issue with upgrading the industry from the form view
has been resolved.

task-3775729
@arpa-odoo arpa-odoo force-pushed the 17.0-base_import_module-fix-activate-arpa branch from b3646a3 to 04f14f8 Compare March 27, 2024 10:28
@vava-odoo vava-odoo marked this pull request as ready for review March 27, 2024 10:48
@C3POdoo C3POdoo requested review from a team, xmo-odoo and HydrionBurst and removed request for a team March 27, 2024 10:52
@vava-odoo vava-odoo requested a review from pimodoo March 27, 2024 13:37
@pimodoo
Copy link
Collaborator

pimodoo commented Mar 27, 2024

@robodoo r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Mar 27, 2024

Merge method set to rebase and fast-forward.

@robodoo robodoo closed this in 5df277f Mar 27, 2024
@fw-bot fw-bot deleted the 17.0-base_import_module-fix-activate-arpa branch April 10, 2024 17:47
willylohws pushed a commit to willylohws/odoo that referenced this pull request May 1, 2024
Before this commit
==================
Unable to activate the industry from the form view in the industry section.
Additionally, the button in the form for upgrading would be called the
button_immediate_upgrade method, resulting in errors.

After this commit
=================
Activation of the industry from the form view in the industry section is now
possible. Furthermore, the issue with upgrading the industry from the form view
has been resolved.

task-3775729

closes odoo#157008

Signed-off-by: Pierre Masereel (pim) <pim@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants