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

[NEW] Handle system/private messages stream #2180

Merged

Conversation

rafaelks
Copy link
Contributor

@RocketChat/ios

Closes #1509

@rafaelks rafaelks added this to the 3.2.0 milestone Sep 10, 2018
@codecov
Copy link

codecov bot commented Sep 10, 2018

Codecov Report

❗ No coverage uploaded for pull request base (develop@ca67cb7). Click here to learn what that means.
The diff coverage is 61.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #2180   +/-   ##
==========================================
  Coverage           ?   34.82%           
==========================================
  Files              ?      393           
  Lines              ?    18559           
  Branches           ?        0           
==========================================
  Hits               ?     6463           
  Misses             ?    12096           
  Partials           ?        0
Impacted Files Coverage Δ
Rocket.Chat/Models/Message/Message.swift 100% <ø> (ø)
Rocket.Chat/Managers/Socket/SocketManager.swift 71.6% <0%> (ø)
...Managers/Model/MessageManager/MessageManager.swift 28.08% <0%> (ø)
.../MessageManager/MessageManagerSystemMessages.swift 100% <100%> (ø)
...Managers/Model/AuthManager/AuthManagerSocket.swift 26.38% <100%> (ø)

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 ca67cb7...6138644. Read the comment docs.

@rafaelks rafaelks changed the title [NEW][WIP] Handle system/private messages stream [NEW] Handle system/private messages stream Sep 10, 2018
@rafaelks rafaelks merged commit bb44953 into develop Sep 14, 2018
@rafaelks rafaelks deleted the feature/handle_system_messages_websocket_stream.1509 branch September 14, 2018 19:42
@rafaelks rafaelks modified the milestones: 3.2.0, 3.1.1 Sep 21, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant