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] portal: portal access management wizard for company #31344

Closed
wants to merge 1 commit into
base: 11.0
from

Conversation

Projects
None yet
4 participants
@kebeclibre
Copy link
Contributor

kebeclibre commented Feb 22, 2019

Have a partner which is a company, with some children
Click on the action "Portal Access Management"

Before this commit, only the children of the company is selectable in the wizard
but not the company itself

After this commit, the company is part of the list

OPW 1931920

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

[FIX] portal: portal access management wizard for company
Have a partner which is a company, with some children
Click on the action "Portal Access Management"

Before this commit, only the children of the company is selectable in the wizard
but not the company itself

After this commit, the company is part of the list

OPW 1931920

@kebeclibre kebeclibre requested a review from nim-odoo Feb 22, 2019

@robodoo robodoo added the seen 🙂 label Feb 22, 2019

@C3POdoo C3POdoo added the OE label Feb 22, 2019

@robodoo robodoo added the CI 🤖 label Feb 22, 2019

@kebeclibre

This comment has been minimized.

Copy link
Contributor Author

kebeclibre commented Feb 22, 2019

robodoo r+

@robodoo robodoo added the r+ 👌 label Feb 22, 2019

robodoo pushed a commit that referenced this pull request Feb 22, 2019

[FIX] portal: portal access management wizard for company
Have a partner which is a company, with some children
Click on the action "Portal Access Management"

Before this commit, only the children of the company is selectable in the wizard
but not the company itself

After this commit, the company is part of the list

OPW 1931920

closes #31344
@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Feb 22, 2019

Staging failed: ci/runbot on 4aa3c54c0bb390d027916dd7daea0822a3950b17 (view more at http://runbot.odoo.com/runbot/build/458630)

@kebeclibre

This comment has been minimized.

Copy link
Contributor Author

kebeclibre commented Feb 22, 2019

robodoo retry r+

@robodoo robodoo added CI 🤖 r+ 👌 and removed error 🙅 labels Feb 22, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Feb 22, 2019

I'm sorry, @kebeclibre. This PR is already reviewed, reviewing it again is useless.

@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Feb 22, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.