Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Vector 443 #118

Merged
merged 5 commits into from
Aug 12, 2016
Merged

Vector 443 #118

merged 5 commits into from
Aug 12, 2016

Conversation

giomfo
Copy link
Member

@giomfo giomfo commented Aug 12, 2016

Redacting membership events should immediately reset the displayname & avatar of room members.

giomfo added 4 commits July 16, 2016 00:40
…& avatar of room members.

element-hq/element-ios#443

- MXEventTimeline: prepare state event redaction.
- MXRoom Notification break: rename "kMXRoomSyncWithLimitedTimelineNotification" with "kMXRoomDidFlushMessagesNotification"
…& avatar of room members.

element-hq/element-ios#443

- Fix: The redacted state event was ignored in room state handling
- Add FIXME related to the blocker issue on server side SYN-724 ("prev_content is totally pruned by the server on redacted membership event").
…& avatar of room members.

element-hq/element-ios#443

- remove FIXME (the blocker issue SYN-724 has been fixed on server side).
@@ -19,6 +19,8 @@
#import "MXSession.h"
#import "MXMemoryStore.h"

#import "MXEvent+MatrixKit.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

Matrix SDK must not depend on the Matrix Kit

giomfo added a commit to matrix-org/matrix-ios-kit that referenced this pull request Aug 12, 2016
giomfo added a commit to element-hq/element-ios that referenced this pull request Aug 12, 2016
@manuroe
Copy link
Contributor

manuroe commented Aug 12, 2016

LGTM

@giomfo giomfo merged commit 33a4a85 into develop Aug 12, 2016
@giomfo giomfo deleted the vector_443 branch August 12, 2016 15:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants