Skip to content

Conversation

@rique223
Copy link
Contributor

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

Created the /featured endpoints on the rest.js file. Also created the necessary typings to use together with it.
@rique223 rique223 requested a review from matheuslc July 29, 2022 19:08
@ggazzo ggazzo added the stat: ready to merge PR tested and approved waiting for merge label Jul 29, 2022
@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #26416 (535b683) into develop (1da6c9e) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             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     
Flag Coverage Δ
e2e 39.05% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
apps/meteor/client/hooks/useTimezoneTime.ts 0.00% <0.00%> (-76.93%) ⬇️
apps/meteor/client/hooks/useUTCClock.ts 33.33% <0.00%> (-66.67%) ⬇️
...app/ui-message/client/popup/messagePopupChannel.js 50.00% <0.00%> (-50.00%) ⬇️
apps/meteor/app/ui-utils/client/lib/RoomManager.js 76.00% <0.00%> (-4.00%) ⬇️
apps/meteor/app/ui-login/client/login/form.js 73.20% <0.00%> (-0.66%) ⬇️
apps/meteor/app/ui/client/views/app/room.js 49.38% <0.00%> (-0.62%) ⬇️
...ached-collection/client/models/CachedCollection.js 85.41% <0.00%> (-0.53%) ⬇️
...teor/app/ui-utils/client/lib/RoomHistoryManager.js 51.44% <0.00%> (-0.49%) ⬇️
apps/meteor/client/hooks/useUpdateAvatar.ts 48.57% <0.00%> (ø)
...ient/views/room/contextualBar/UserInfo/UserInfo.js
... and 12 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

@rique223 rique223 marked this pull request as ready for review July 29, 2022 21:33
@rique223 rique223 requested review from a team as code owners July 29, 2022 21:33
thassiov
thassiov previously approved these changes Aug 1, 2022
Copy link
Contributor

@thassiov thassiov left a 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.
Copy link
Contributor

@graywolf336 graywolf336 left a comment

Choose a reason for hiding this comment

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

Great work!

@rique223 rique223 requested a review from thassiov August 2, 2022 17:43
@graywolf336 graywolf336 removed the request for review from matheuslc August 2, 2022 23:50
@casalsgh casalsgh requested a review from ggazzo August 5, 2022 14:47
@ggazzo ggazzo removed their request for review August 5, 2022 14:57
@ggazzo ggazzo changed the title [NEW] Surface featured apps endpoint to RC [NEW] Surface featured apps endpoint Aug 5, 2022
@kodiakhq kodiakhq bot merged commit 28ee94b into develop Aug 5, 2022
@kodiakhq kodiakhq bot deleted the feat/surface-featured-apps-endpoint branch August 5, 2022 15:05
gabriellsh added a commit that referenced this pull request Aug 8, 2022
…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)
gabriellsh added a commit that referenced this pull request Aug 8, 2022
…/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)
gabriellsh added a commit that referenced this pull request Aug 8, 2022
…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)
  ...
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants