Closed Bug 1095533 Opened 10 years ago Closed 10 years ago

Add a boolean 'owner' field to the participant object when it's the owner of that room

Categories

(Hello (Loop) :: Server, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mikedeboer, Assigned: rhubscher)

References

Details

Attachments

(1 file)

Snippet of my IRC conversation with Rémy: [15:22:00] <natim> mreavy: mikedeboer actually you can use the participant account property for that [15:22:39] <mikedeboer> natim: so the participant that has an account property is the current owner? [15:23:06] <natim> You mean for other people or for yourself? [15:23:17] <natim> Who needs to know it is the owner? [15:24:18] <natim> If you are not the owner you don't know who is the owner except if you get the room and see who is the roomOwner [15:24:59] <mikedeboer> natim: wait, you only get the rooms you own in GET rooms/, right? [15:25:19] <natim> On GET /rooms yet but not on GET /rooms/:token [15:25:27] <natim> https://wiki.mozilla.org/index.php?title=Loop/Architecture/Rooms#GET_.2Frooms.2F.7Btoken.7D [15:25:41] <mikedeboer> natim: right! that anwers my question :) [15:25:44] <mikedeboer> thanks! [15:25:50] <natim> Also the doc is wrong [15:25:52] <natim> fixing [15:26:53] <natim> mikedeboer: let me double check that actually [15:28:15] <natim> mikedeboer: you are right there is something here [15:28:41] <natim> Actually the room owner gives the roomOwner content during the roomCreation [15:28:56] <natim> https://wiki.mozilla.org/Loop/Architecture/Rooms#POST_.2Frooms [15:29:55] <mikedeboer> ok [15:29:56] <natim> We should probably add a owner property on the participant if he is the owner [15:30:11] <natim> Could you please add a bug about that so we can discuss it with Adam? [15:30:12] <mikedeboer> natim: that actually makes a lot of sense to me [15:30:22] <mikedeboer> natim: sure [15:30:30] <natim> Thank you I reckon we'd like to have this for Fx 35.
Assignee: nobody → rhubscher
Status: NEW → ASSIGNED
Attached file Link to GitHub PR.
Attachment #8522321 - Flags: review?(tarek)
Attachment #8522321 - Flags: review?(tarek) → review+
Blocks: 1074666
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: