Skip to content
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

Chore: convert invites, misc and subscriptions to TS and create definitions #25350

Merged
merged 36 commits into from
Jun 7, 2022

Conversation

felipe-rod123
Copy link
Contributor

Proposed changes (including videos or screenshots)

Converted apps/meteor/app/api/server/v1/invites.js, misc.js and subscriptions.js to Typescript and created their endpoint definitions on the rest-typings folder.

Issue(s)

Steps to test or reproduce

Further comments

@felipe-rod123 felipe-rod123 requested a review from a team as a code owner April 30, 2022 03:16
@ggazzo ggazzo force-pushed the chore/convert-ts-and-create-endpoints branch from e7042b4 to 89017c8 Compare May 24, 2022 02:32
Copy link
Member

@ggazzo ggazzo left a comment

Choose a reason for hiding this comment

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

Please create the json schema for each endpoint

@ggazzo ggazzo force-pushed the chore/convert-ts-and-create-endpoints branch from bfa1e86 to 20487b1 Compare May 25, 2022 16:42
@ggazzo ggazzo force-pushed the chore/convert-ts-and-create-endpoints branch from 20487b1 to bc0bf0c Compare May 25, 2022 16:53
@felipe-rod123 felipe-rod123 requested a review from a team as a code owner May 30, 2022 12:19
@lgtm-com
Copy link

lgtm-com bot commented Jun 4, 2022

This pull request introduces 1 alert when merging c65b728 into 81304fe - view on LGTM.com

new alerts:

  • 1 for Syntax error

@ggazzo ggazzo force-pushed the chore/convert-ts-and-create-endpoints branch from 313d05e to 6ce41de Compare June 7, 2022 04:44
@ggazzo ggazzo force-pushed the chore/convert-ts-and-create-endpoints branch from 6ce41de to bb350be Compare June 7, 2022 04:54
@ggazzo ggazzo force-pushed the chore/convert-ts-and-create-endpoints branch from 8f69518 to 8e94144 Compare June 7, 2022 14:43
Co-authored-by: Pierre Lehnen <55164754+pierre-lehnen-rc@users.noreply.github.com>
Co-authored-by: Pierre Lehnen <55164754+pierre-lehnen-rc@users.noreply.github.com>
@ggazzo ggazzo merged commit afcbf1c into develop Jun 7, 2022
@ggazzo ggazzo deleted the chore/convert-ts-and-create-endpoints branch June 7, 2022 20:12
gabriellsh added a commit that referenced this pull request Jun 8, 2022
…homepage

* 'develop' of github.com:RocketChat/Rocket.Chat:
  Chore: Fix CI (#25797)
  [FIX] Fix prom-client new promise usage (#25781)
  [FIX] Discussion alphabetical  ordering (#25788)
  Update .kodiak.toml
  Update .kodiak.toml
  Chore: Testing Kodiak feature (#25794)
  [FIX] Broken Omnichannel>Agents page (#25731)
  [FIX] Sanitize styles in message (#25744)
  chore: Convert to TS RoomAutoComplete (#25536)
  Chore: API test on method GET with params as a number. (#25769)
  Chore: convert invites, misc and subscriptions to TS and create definitions (#25350)
  Chore: Remove .snapcraft
  Chore: Remove toastr package (#25787)
  [BREAK] Remove Blockstack authentication (#25649)
  [FIX] getUserMentionsByChannel method room permission (#25748)
  [NEW] Fuselage ToastBar (#25583)
gabriellsh added a commit that referenced this pull request Jun 8, 2022
… feat/custom-homepage-setting

* 'new/homepage' of github.com:RocketChat/Rocket.Chat:
  Chore: Fix CI (#25797)
  [FIX] Fix prom-client new promise usage (#25781)
  [FIX] Discussion alphabetical  ordering (#25788)
  Update .kodiak.toml
  Update .kodiak.toml
  Chore: Testing Kodiak feature (#25794)
  [FIX] Broken Omnichannel>Agents page (#25731)
  [FIX] Sanitize styles in message (#25744)
  chore: Convert to TS RoomAutoComplete (#25536)
  Chore: API test on method GET with params as a number. (#25769)
  Chore: convert invites, misc and subscriptions to TS and create definitions (#25350)
  Chore: Remove .snapcraft
  Chore: Remove toastr package (#25787)
  [BREAK] Remove Blockstack authentication (#25649)
  [FIX] getUserMentionsByChannel method room permission (#25748)
  [NEW] Fuselage ToastBar (#25583)
gabriellsh added a commit that referenced this pull request Jun 9, 2022
… feat/new-header

* 'new/homepage' of github.com:RocketChat/Rocket.Chat:
  [NEW] Add Custom homepage Content Setting (#25777)
  Chore: Fix CI (#25797)
  [FIX] Fix prom-client new promise usage (#25781)
  [FIX] Discussion alphabetical  ordering (#25788)
  Update .kodiak.toml
  Update .kodiak.toml
  Chore: Testing Kodiak feature (#25794)
  [FIX] Broken Omnichannel>Agents page (#25731)
  [FIX] Sanitize styles in message (#25744)
  chore: Convert to TS RoomAutoComplete (#25536)
  Chore: API test on method GET with params as a number. (#25769)
  Chore: convert invites, misc and subscriptions to TS and create definitions (#25350)
  Chore: Remove .snapcraft
  Chore: Remove toastr package (#25787)
  [BREAK] Remove Blockstack authentication (#25649)
  [FIX] getUserMentionsByChannel method room permission (#25748)
  [NEW] Fuselage ToastBar (#25583)
@murtaza98 murtaza98 mentioned this pull request Jul 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants