Skip to content

Comments

[FIX] Implement client errors on ddp-streamer#24310

Merged
sampaiodiego merged 18 commits intodevelopfrom
implement-client-errors-ms
Feb 10, 2022
Merged

[FIX] Implement client errors on ddp-streamer#24310
sampaiodiego merged 18 commits intodevelopfrom
implement-client-errors-ms

Conversation

@sampaiodiego
Copy link
Member

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Created as draft to discuss about a few things:

  • should this PR go through all errors thrown from existing services and change them to use one of the new error types?
  • open a discussion regarding the new error type names, currently MethodError and ClientSafeError

@sampaiodiego sampaiodiego requested a review from a team January 27, 2022 14:46
@lgtm-com
Copy link

lgtm-com bot commented Jan 28, 2022

This pull request introduces 1 alert when merging 5eb6a6e into 042bde1 - view on LGTM.com

new alerts:

  • 1 for Invocation of non-function

@sampaiodiego sampaiodiego marked this pull request as ready for review February 4, 2022 12:01
@sampaiodiego sampaiodiego merged commit 4fdd558 into develop Feb 10, 2022
@sampaiodiego sampaiodiego deleted the implement-client-errors-ms branch February 10, 2022 19:53
gabriellsh added a commit that referenced this pull request Feb 15, 2022
…age-template-2

* 'develop' of github.com:RocketChat/Rocket.Chat: (29 commits)
  Bump vm2 from 3.9.5 to 3.9.7 in /ee/server/services (#24509)
  [IMPROVE] ChatBox Text to File Description (#24451)
  Chore: Update Meteor to 2.5.6 (#24461)
  Chore: Update ws package (#24477)
  Bump underscore.string from 3.3.5 to 3.3.6 in /ee/server/services (#24498)
  Bump follow-redirects from 1.14.7 to 1.14.8 in /ee/server/services (#24491)
  Language update from LingoHub 🤖 (#24493)
  Covert to typescript the unarchive slash commands files (#24331)
  [IMPROVE] Add tooltips on action buttons of Canned Response message composer (#24483)
  Chore: Delete unused file (NewAdminInfoPage.js) (#24196)
  [IMPROVE][ENTERPRISE] Improve how micro services are loaded (#24388)
  [IMPROVE] Add return button in chats opened from the list of current chats (#24458)
  Bump express from 4.17.1 to 4.17.2 in /ee/server/services (#24469)
  Bump cookie from 0.4.1 to 0.4.2 in /ee/server/services (#24472)
  [IMPROVE] Close modal on esc and outside click (#24275)
  Chore(deps-dev): Bump ts-node in /ee/server/services (#24435)
  [IMPROVE] Add user to room on "Click to Join!" button press (#24041)
  [FIX] Implement client errors on ddp-streamer (#24310)
  Bump body-parser from 1.19.0 to 1.19.1 in /ee/server/services (#23963)
  Bump jaeger-client from 3.18.1 to 3.19.0 in /ee/server/services (#23961)
  ...
@pierre-lehnen-rc pierre-lehnen-rc mentioned this pull request Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants