-
Notifications
You must be signed in to change notification settings - Fork 11.3k
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
[NEW] Surface featured apps endpoint #26416
Conversation
Created the /featured endpoints on the rest.js file. Also created the necessary typings to use together with it.
Codecov Report
@@ Coverage Diff @@
## develop #26416 +/- ##
===========================================
+ Coverage 39.02% 39.05% +0.03%
===========================================
Files 753 748 -5
Lines 18764 18647 -117
Branches 1414 1424 +10
===========================================
- Hits 7322 7283 -39
+ Misses 11224 11141 -83
- Partials 218 223 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixed a problem in which the featured endpoint would be out prioritized by the rest client because it was bellow a wildcard endpoint. Also solved a typo on the route definition for the featured endpoint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
…ove-e2ee-threadlist * 'develop' of github.com:RocketChat/Rocket.Chat: Chore: Remove settings Fibers usage (#26465) Chore: Refactor create-target-channel util (#26493) Chore: useEndpointData deprecation (#26494) [NEW] Surface featured apps endpoint (#26416) [FIX] Request at least one field in the payload of `/v1/users.setStatus` (#26490) Chore: Exit process on `unhandledRejection` on CI (#26467) [FIX] Clear push token on save user password (#26466)
…/not-found-page * 'develop' of github.com:RocketChat/Rocket.Chat: Chore: Remove settings Fibers usage (#26465) Chore: Refactor create-target-channel util (#26493) Chore: useEndpointData deprecation (#26494) [NEW] Surface featured apps endpoint (#26416) [FIX] Request at least one field in the payload of `/v1/users.setStatus` (#26490) Chore: Exit process on `unhandledRejection` on CI (#26467) [FIX] Clear push token on save user password (#26466) [FIX] Undefined MediaDevices error on HTTP (#26396) Chore: Codecov threshold (#26477)
…homepage * 'develop' of github.com:RocketChat/Rocket.Chat: (39 commits) Chore: ESLint warnings (#26504) Chore: Remove settings Fibers usage (#26465) Chore: Refactor create-target-channel util (#26493) Chore: useEndpointData deprecation (#26494) [NEW] Surface featured apps endpoint (#26416) [FIX] Request at least one field in the payload of `/v1/users.setStatus` (#26490) Chore: Exit process on `unhandledRejection` on CI (#26467) [FIX] Clear push token on save user password (#26466) [FIX] Undefined MediaDevices error on HTTP (#26396) Chore: Codecov threshold (#26477) Chore: Tests intermitences (#26464) [FIX] Don't give errors on outbound voip call Request Terminated (#26373) Chore: Use Docker compose on CI (#26437) [FIX] DialPad call button from end to center (#26459) Chore: Parallelize e2e tests (#26390) [IMPROVE] use enter key to call using DialPad (#26454) Chore: Accounts/token to TS (#26434) Chore: Purge some unused modules (#26447) i18n: Language update from LingoHub 🤖 on 2022-08-01Z (#26429) Chore: Add end-to-end tests to teams listing in the `directory` endpoint (#26347) ...
Proposed changes (including videos or screenshots)
Created the /featured endpoints on the rest.js file. Also created the necessary typings to use together with it.
Issue(s)
Steps to test or reproduce
Further comments