Closed
Bug 1103996
Opened 10 years ago
Closed 10 years ago
[Room][Notifications] Create 'room notifications' channel helper
Categories
(Firefox OS Graveyard :: Gaia::Loop, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: borjasalguero, Assigned: borjasalguero)
References
Details
Attachments
(1 file)
We need a handler in order to listen and handle all events coming from the server related with Rooms. We will be listening and we should offer an interface in order to subscribe to all possible events from outside of the module.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → borja.bugzilla
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8527800 -
Flags: review?(josea.olivera)
Comment 2•10 years ago
|
||
Comment on attachment 8527800 [details] [review]
Pull Request
Looking really good. Have a look at the PR please. Request review at me again once everything gets addressed please. Thanks!
Attachment #8527800 -
Flags: review?(josea.olivera) → feedback+
Assignee | ||
Updated•10 years ago
|
Attachment #8527800 -
Flags: review?(josea.olivera)
Comment 3•10 years ago
|
||
Comment on attachment 8527800 [details] [review]
Pull Request
LGTM. r=me
Still a comment on the PR. Please address it and land at will. Tested a it works fine. Important change but nothing gets broken. Thanks for taking care of it!
Attachment #8527800 -
Flags: review?(josea.olivera) → review+
Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•