Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

[NEW] Support all kind of System Messages #2303

Merged
merged 9 commits into from
Nov 14, 2018

Conversation

rafaelks
Copy link
Contributor

@rafaelks rafaelks commented Nov 10, 2018

@RocketChat/ios

Closes #1167

  • Message Snippeted;
  • Room Changed Privacy;
  • Room Changed Topic;
  • Room Changed Announcement;
  • Room Changed Description;

Jitsi message support will be added in another PR.

@codecov
Copy link

codecov bot commented Nov 10, 2018

Codecov Report

Merging #2303 into develop will increase coverage by 0.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2303      +/-   ##
===========================================
+ Coverage    26.09%   26.11%   +0.01%     
===========================================
  Files          438      438              
  Lines        15933    15969      +36     
===========================================
+ Hits          4158     4170      +12     
- Misses       11775    11799      +24
Impacted Files Coverage Δ
Rocket.Chat/Models/Message/UnmanagedMessage.swift 9.52% <ø> (ø) ⬆️
Rocket.Chat/Models/Message/Message.swift 89.18% <ø> (ø) ⬆️
...cket.Chat/Models/Mapping/MessageModelMapping.swift 83.51% <100%> (+0.37%) ⬆️
...ket.Chat/Extensions/Models/MessageExtensions.swift 41.84% <29.41%> (-3.96%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 638135c...aa9c2ff. Read the comment docs.

@rafaelks rafaelks changed the title [WIP][NEW] Support all kind of System Messages [NEW] Support all kind of System Messages Nov 14, 2018
@rafaelks rafaelks added this to the 3.2.0 milestone Nov 14, 2018
@rafaelks rafaelks merged commit 00dcaa0 into develop Nov 14, 2018
@rafaelks rafaelks deleted the feature/fully_support_custom_message_types.1167 branch November 14, 2018 19:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants