-
Notifications
You must be signed in to change notification settings - Fork 114
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
Services dynamically add their component if they have one in the relevant registry #925
Conversation
c889209
to
bc2915a
Compare
|
||
const { EventBus } = owl.core; | ||
|
||
export const popoverService = { | ||
start() { | ||
let nextId = 0; | ||
/** |
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.
why did you move the docstring?
5685a95
to
544c6c3
Compare
dfc5f12
to
8ae952b
Compare
robodoo rebase-ff r+ |
Merge method set to rebase and fast-forward |
ee7ad7e
to
39c1faa
Compare
@@ -1,22 +1,18 @@ | |||
/** @odoo-module **/ | |||
|
|||
import { registry } from "../registry"; | |||
import { useService } from "../service_hook"; | |||
import { Notification as NotificationComponent } from "./notification"; |
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.
no need to rename it, no?
robodoo r+ |
Unable to stage PR ({"message":"Merge conflict","documentation_url":"https://docs.github.com/rest/reference/repos#merge-a-branch"}) |
Some services are coupled with a Component. Usually the service handles the state of the system, and the Component displays or uses it. To enable the communication between the service and the component while making it private, the services should add themselves their Component in the relevant registry, with the proper means of communication passed in props. This mechanism relies on c1d49d4
39c1faa
to
46b3576
Compare
robodoo retry r+ |
I'm sorry, @ged-odoo. Retry makes no sense when the PR is not in error. |
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
closes #925 Related: odoo-dev/enterprise#166 Signed-off-by: Géry Debongnie (ged) <ged@openerp.com>
No description provided.