Skip to content
This repository has been archived by the owner on Dec 12, 2022. It is now read-only.

Groups: Display flair for users in room history. #395

Merged
merged 1 commit into from
Feb 5, 2018
Merged

Conversation

giomfo
Copy link
Member

@giomfo giomfo commented Feb 4, 2018

MXKRoomDataSource: Force a refresh on flair settings change.
MXKRoomBubbleCellData: keep update the flair of the sender.
MXKRoomBubbleTableViewCell: Display the sender flair if any

TODO: display the full flair list when the user clicks on the displayed flair.

element-hq/riot-meta#118

MXKRoomDataSource: Force a refresh on flair settings change.
MXKRoomBubbleCellData: keep update the flair of the sender.
MXKRoomBubbleTableViewCell: Display the sender flair if any

TODO: display the full flair list when the user clicks on the displayed flair.

element-hq/riot-meta#118
@giomfo giomfo requested a review from manuroe February 4, 2018 16:11
Copy link
Contributor

@manuroe manuroe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
@giomfo, could you just check my 2 questions, please?

}

// Observe any change on publicised groups for the message sender
[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(didMXSessionUpdatePublicisedGroupsForUsers:) name:kMXSessionDidUpdatePublicisedGroupsForUsersNotification object:self.mxSession];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a mechanism to refresh the sender flair if the room has been added to a group while the app was in background?
I ask for this particular use case because of the group API design, we only update group data only when the app comes back to foreground.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@manuroe: the rooms added to a group are ignored for the moment in the flair process.
Flair takes into account only the related groups defined in room settings and the publicised groups in the user's profile.

if (MXTimelineDirectionForwards == direction)
{
// The flair settings have been updated: flush the current bubble data and rebuild them.
[self reload];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it the answer to my question in the other comment?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really, here we detect that the related groups have been updated. We reload the data source because all the room bubbles have to update their sender flair.

@giomfo giomfo merged commit 7f854e2 into develop Feb 5, 2018
@giomfo giomfo deleted the meta_118 branch February 5, 2018 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants