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

[IMP] web: control panel add on_detach_callback #30758

Closed

Conversation

rfr-odoo
Copy link
Contributor

@rfr-odoo rfr-odoo commented Feb 1, 2019

Adding event listener in on_attach_callback need to be removed
in on_detach_callback.

Before this commit on_detach_callback was not implemented in control panel.
After this commit we can use on_detach_callback.

Needed for task ID: 1934261

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

Adding event listener in on_attach_callback need to be removed
in on_detach_callback.

Before this commit on_detach_callback was not implemented in control panel.
After this commit we can use on_detach_callback.

Needed for task ID: 1934261
@C3POdoo C3POdoo added the RD research & development, internal work label Feb 1, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 1, 2019
@adr-odoo
Copy link
Contributor

adr-odoo commented Feb 8, 2019

@robodoo r+

@robodoo robodoo added the r+ 👌 label Feb 8, 2019
@robodoo
Copy link
Contributor

robodoo commented Feb 8, 2019

Linked pull request(s) odoo/enterprise#3538 not ready. Linked PRs are not staged until all of them are ready.

robodoo pushed a commit that referenced this pull request Feb 8, 2019
Adding event listener in on_attach_callback need to be removed
in on_detach_callback.

Before this commit on_detach_callback was not implemented in control panel.
After this commit we can use on_detach_callback.

Needed for task ID: 1934261

closes #30758
@robodoo
Copy link
Contributor

robodoo commented Feb 8, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 8, 2019
@adr-odoo adr-odoo deleted the master-test-control-panel-mobile-rfr branch February 8, 2019 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants