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

[NEW] Add support to "Unread Marker" on the list of messages #2262

Merged

Conversation

rafaelks
Copy link
Contributor

@RocketChat/ios

This PR adds support to unread marker on the list and also marks the subscription as read when view appear.

simulator screen shot - iphone se - 2018-10-25 at 09 56 41

@rafaelks rafaelks added this to the 3.2.0 milestone Oct 25, 2018
@rafaelks rafaelks added this to In Progress in Cycle 10: 10/05 ~ 11/05 via automation Oct 25, 2018
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Loading configuration from '.swiftlint.yml'
Loading configuration from '.swiftlint.yml'
Linting Swift files at paths Rocket.Chat/Views/Chat/New, Chat/Cells/UnreadMarkerCell.swift
No lintable files found at paths: 'Rocket.Chat/Views/Chat/New, Chat/Cells/UnreadMarkerCell.swift'

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Loading configuration from '.swiftlint.yml'
Loading configuration from '.swiftlint.yml'
Linting Swift files at paths Rocket.Chat/Views/Chat/New, Chat/Cells/UnreadMarkerCell.swift
No lintable files found at paths: 'Rocket.Chat/Views/Chat/New, Chat/Cells/UnreadMarkerCell.swift'

@codecov
Copy link

codecov bot commented Oct 25, 2018

Codecov Report

Merging #2262 into feature/integrate-rc-view-controller will decrease coverage by 0.02%.
The diff coverage is 4.16%.

Impacted file tree graph

@@                           Coverage Diff                            @@
##           feature/integrate-rc-view-controller    #2262      +/-   ##
========================================================================
- Coverage                                 25.73%   25.71%   -0.03%     
========================================================================
  Files                                       434      436       +2     
  Lines                                     16341    16363      +22     
========================================================================
+ Hits                                       4206     4207       +1     
- Misses                                    12135    12156      +21
Impacted Files Coverage Δ
Rocket.Chat/Extensions/UIColorExtension.swift 56.81% <ø> (ø) ⬆️
...ket.Chat/Controllers/Chat/ChatViewController.swift 0% <ø> (ø) ⬆️
...t/Views/Chat/New Chat/Cells/UnreadMarkerCell.swift 0% <0%> (ø)
...cket.Chat/Controllers/Chat/MessagesViewModel.swift 20.51% <0%> (-0.27%) ⬇️
...Controllers/Chat/ChatSections/MessageSection.swift 1.02% <0%> (-0.02%) ⬇️
...Chat/New Chat/ChatItems/UnreadMarkerChatItem.swift 0% <0%> (ø)
Rocket.Chat/API/Clients/SubscriptionsClient.swift 37.97% <0%> (ø) ⬆️
...scriptionManager/SubscriptionManagerMessages.swift 0% <0%> (ø) ⬆️
...Chat/Controllers/Chat/MessagesViewController.swift 43.05% <25%> (-1.07%) ⬇️
... and 1 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 09dbb12...418ce83. Read the comment docs.

Cycle 10: 10/05 ~ 11/05 automation moved this from In Progress to In Review / QA Oct 25, 2018
Copy link
Collaborator

@filipealva filipealva left a comment

Choose a reason for hiding this comment

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

💯

@filipealva filipealva merged commit 171e242 into feature/integrate-rc-view-controller Oct 25, 2018
Cycle 10: 10/05 ~ 11/05 automation moved this from In Review / QA to Done Oct 25, 2018
@filipealva filipealva deleted the feature/unread_marker branch October 25, 2018 14:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants