Skip to content
This repository was archived by the owner on Oct 30, 2025. It is now read-only.

[NEW] Add subscriptions.unread endpoint#228

Merged
philipbrito merged 2 commits intoRocketChat:developfrom
shubhsherl:n_1055
Dec 21, 2018
Merged

[NEW] Add subscriptions.unread endpoint#228
philipbrito merged 2 commits intoRocketChat:developfrom
shubhsherl:n_1055

Conversation

@shubhsherl
Copy link
Copy Markdown
Contributor

This PR adds a function which is required in Rocket.Chat.Android#1929

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Dec 19, 2018

CLA assistant check
All committers have signed the CLA.

data class ChatRoomPayload(@Json(name = "rid") val roomId: String)

@JsonSerializable
data class ChatRoomUnreadPayload(@Json(name = "roomId") val roomId: String)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since the property here is declared like the expected JSON key name you don't need to use this annotation here.

}
}

@Test
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for adding the tests! 💯

@philipbrito philipbrito changed the title add subscriptions.unread [NEW] Add subscriptions.unread endpoint Dec 21, 2018
@philipbrito
Copy link
Copy Markdown
Contributor

@shubhsherl Could you assign our CLA please?

Copy link
Copy Markdown
Contributor

@philipbrito philipbrito left a comment

Choose a reason for hiding this comment

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

Thanks!

@philipbrito philipbrito merged commit bea2945 into RocketChat:develop Dec 21, 2018
@shubhsherl shubhsherl deleted the n_1055 branch December 24, 2018 17:09
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