Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore: Add @rocket.chat/rest-typings #6232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
diegolmello opened this issue Mar 12, 2025 · 1 comment
Open

chore: Add @rocket.chat/rest-typings #6232

diegolmello opened this issue Mar 12, 2025 · 1 comment

Comments

@diegolmello
Copy link
Member

diegolmello commented Mar 12, 2025

When this repo was migrated to TS, we didn't have any source of truth to get REST API definitions from, so we just duplicated typings from the backend and adapted them a little bit to our repo.

https://github.com/RocketChat/Rocket.Chat.ReactNative/tree/develop/app/definitions

Let's refactor that to use this instead https://www.npmjs.com/package/@rocket.chat/rest-typings

I can't emphasize how important this work is and how much it's going to improve our app maintainability and readability.

Some work was done here, but the scope was bigger #5918
Let's focus on this issue first and then we can focus that PR on migrating the SDK.

@Demiserular
Copy link

@diegolmello
I would like to work on this one .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants