-
Notifications
You must be signed in to change notification settings - Fork 212
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
Replies: Implement sending #534
Conversation
…tring localizations when send reply to a message in a room.
…ult localization strings used when send reply event to a message in a room.
…pted content when encrypting an event.
…clear content when decrypting an event.
…d content when encrypting an event. Fix send reply formatted body building when input formatted body is nil.
…ge. Add a better check for `m.relates_to` content field validity.
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.
LGTM except 2 small remarks
MatrixSDK/Data/MXRoom.m
Outdated
{ | ||
if (eventToReply.eventType != MXEventTypeRoomMessage) | ||
{ | ||
return false; |
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 for consistency.
MatrixSDKTests/MXRoomTests.m
Outdated
NSString *expectedSecondEventBody = [NSString stringWithFormat:expectedSecondEventBodyStringFormat, firstEventSender]; | ||
NSString *expectedSecondEventFormattedBody = [NSString stringWithFormat:expectedSecondEventFormattedBodyStringFormat, permalinkToEvent, permalinkToUser, firstEventSender]; | ||
|
||
XCTAssert([secondEventBody isEqualToString:expectedSecondEventBody]); |
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.
Use XCTAssertEqualObjects to compare strings. This is helpful to debug remotely.
Implement reply to an event with text message.
element-hq/element-ios#1911