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

Conversation

@rafaelks
Copy link
Contributor

@RocketChat/ios

DM

img_0258

Group, 1 user typing

img_0255

Group, 2 users typing

img_0256

Group, 3+ users typing

img_0257

@rafaelks rafaelks added this to the 3.2.0 milestone Nov 14, 2018

extension SubscriptionManager {
static func subscribeTypingEvent(_ subscription: Subscription, completion: @escaping (String?, Bool) -> Void) {
static func subscribeTypingEvent(_ subscription: UnmanagedSubscription, completion: @escaping (String?, Bool) -> Void) {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 124 characters (line_length)

}
}

private static func websocketUpdatePublicSettings(_ realm: Realm?, _ auth: Auth?, completion: (MessageCompletionObject<AuthSettings>)? = nil) {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 147 characters (line_length)

completion?(unmanagedSettings)
}

static func updatePublicSettings(serverVersion: Version? = nil, apiHost: URL? = nil, _ auth: Auth?, completion: (MessageCompletionObject<AuthSettings>)? = nil) {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 165 characters (line_length)


extension AuthSettingsManager {

static func persistPublicSettings(settings: AuthSettings, realm: Realm? = Realm.current, completion: (MessageCompletionObject<AuthSettings>)? = nil) {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 154 characters (line_length)

Copy link
Member

@cardoso cardoso left a comment

Choose a reason for hiding this comment

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

Cool! 😎

@codecov
Copy link

codecov bot commented Nov 14, 2018

Codecov Report

Merging #2314 into develop will decrease coverage by 0.19%.
The diff coverage is 35.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop    #2314     +/-   ##
==========================================
- Coverage    26.11%   25.92%   -0.2%     
==========================================
  Files          438      439      +1     
  Lines        15969    15998     +29     
==========================================
- Hits          4170     4147     -23     
- Misses       11799    11851     +52
Impacted Files Coverage Δ
Rocket.Chat/Models/User/UserUtils.swift 100% <ø> (+5.26%) ⬆️
...at/Models/Subscription/UnmanagedSubscription.swift 0% <ø> (ø) ⬆️
Rocket.Chat/Models/User/UnmanagedUser.swift 0% <ø> (ø) ⬆️
...cket.Chat/Managers/Model/AuthSettingsManager.swift 75% <ø> (+12.5%) ⬆️
...ubscriptionManager/SubscriptionManagerTyping.swift 0% <0%> (ø) ⬆️
...t.Chat/Models/Subscription/SubscriptionUtils.swift 31.64% <0%> (-5.14%) ⬇️
Rocket.Chat/Views/Chat/ChatTitleViewModel.swift 100% <100%> (ø) ⬆️
Rocket.Chat/Views/Chat/ChatTitleView.swift 38.7% <15.78%> (-36.3%) ⬇️
...odel/AuthSettingsManagerPersistencyExtension.swift 25% <25%> (ø)
...Chat/Controllers/Chat/MessagesViewController.swift 36.84% <50%> (-0.8%) ⬇️
... and 8 more

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 00dcaa0...6ebc29b. Read the comment docs.

@rafaelks rafaelks merged commit 7daccd8 into develop Nov 14, 2018
@rafaelks rafaelks deleted the feature/typing_status branch November 14, 2018 20:04
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.

4 participants