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

[NEW] Allow replying in notification (Quick Reply) #1023

Merged
merged 8 commits into from
Dec 4, 2017

Conversation

cardoso
Copy link
Collaborator

@cardoso cardoso commented Nov 30, 2017

@RocketChat/ios

Description

This feature allows the user to reply to a message from a notification without having to enter the App. (iOS 10 capability)

Will only work in Rocket.Chat servers with 0.60.0 or above.
Here's an instance for testing: https://unstable.rocket.chat

Closes #831

Preview

Progress

  • Add reply action to eligible notifications
  • Reply using REST API
  • Mention user if it's not a DM
  • Tests

}
}

Choose a reason for hiding this comment

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

Trailing Newline Violation: Files should have a single trailing newline. (trailing_newline)

@cardoso cardoso changed the title [WIP][NEW] Allow replying from notification [WIP][NEW] Allow replying in notification (Quick Reply) Nov 30, 2017
@cardoso cardoso added this to the 1.8.0 milestone Nov 30, 2017
@cardoso
Copy link
Collaborator Author

cardoso commented Dec 1, 2017

I wanted to also quote the message like we do when replying from inside the App, but for now we are blocked by RocketChat/Rocket.Chat#8996. No big deal I guess.

@codecov
Copy link

codecov bot commented Dec 3, 2017

Codecov Report

Merging #1023 into develop will increase coverage by 0.23%.
The diff coverage is 86.79%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1023      +/-   ##
===========================================
+ Coverage    38.23%   38.46%   +0.23%     
===========================================
  Files          220      220              
  Lines        13051    13094      +43     
  Branches       691      695       +4     
===========================================
+ Hits          4990     5037      +47     
+ Misses        7611     7608       -3     
+ Partials       450      449       -1
Impacted Files Coverage Δ
Rocket.ChatTests/Managers/PushManagerSpec.swift 88.88% <100%> (ø) ⬆️
Rocket.Chat/Managers/PushManager.swift 58.08% <86.27%> (+12.92%) ⬆️
Rocket.ChatTests/Models/UserSpec.swift 89.53% <0%> (ø) ⬆️
Rocket.Chat/API/API.swift 43.47% <0%> (+21.73%) ⬆️

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 0df8fcd...9b7470c. Read the comment docs.

@cardoso cardoso changed the title [WIP][NEW] Allow replying in notification (Quick Reply) [NEW] Allow replying in notification (Quick Reply) Dec 3, 2017
@cardoso cardoso requested a review from rafaelks December 3, 2017 15:56
@rafaelks rafaelks merged commit ebc0799 into develop Dec 4, 2017
@rafaelks
Copy link
Contributor

rafaelks commented Dec 4, 2017

Looking very very good! 🎉

@rafaelks rafaelks deleted the feat/notification_reply.831 branch December 4, 2017 09:14
@rafaelks rafaelks modified the milestones: 1.8.0, 1.7.4 Dec 4, 2017
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.

3 participants