Skip to content

Conversation

@aditya-mitra
Copy link
Contributor

Proposed changes (including videos or screenshots)

Added explicit Return Type (Promise) on the function to fix eslint warning (explicit-function-return-type)

Issue(s)

Steps to test or reproduce

On the terminal (in the rocketchat folder), run command npm run tslint

Screenshot-20210127-013358

Further comments

It was always thought I had something wrong in my code until I had a closer look to it.

@ggazzo ggazzo changed the title [FIX] ESLint Warning - explicit-function-return-type Regression: ESLint Warning - explicit-function-return-type Jan 30, 2021
@ggazzo ggazzo requested a review from a team January 30, 2021 02:56
@ggazzo ggazzo merged commit 0d9afda into RocketChat:develop Jan 30, 2021
@sampaiodiego sampaiodiego mentioned this pull request Feb 1, 2021
@aditya-mitra aditya-mitra deleted the aditya-mitra-eslint-fix branch February 3, 2021 15:33
vanhoang1107 pushed a commit to vanhoang1107/Rocket.Chat that referenced this pull request Feb 5, 2021
* rocketchat/master: (133 commits)
  Bump version to 3.11.0
  Bump version to 3.11.0-rc.7
  Bump version to 3.11.0-rc.6
  Regression: Set image sizes based on rotation (RocketChat#20531)
  Regression: Apps-Engine - Convert streams to buffers on file upload (RocketChat#20523)
  Regression: Room not scrolling to bottom (RocketChat#20516)
  Regression: NPS (RocketChat#20514)
  Regression: Fix e2e paused state (RocketChat#20511)
  Regression: Custom field labels are not displayed properly on Omnichannel Contact Profile form (RocketChat#20393)
  Bump version to 3.11.0-rc.5
  Regression: Fix banners sync data types (RocketChat#20517)
  Regression: Fixed update room avatar issue.  (RocketChat#20433)
  Regression: ESLint Warning - explicit-function-return-type (RocketChat#20434)
  Bump version to 3.11.0-rc.4
  [FIX][Apps] Don't show the "review permissions" modal when there's none to review (RocketChat#20506)
  Revert "[NEW][ENTERPRISE] Hide message history for new channel members (RocketChat#20253)" (RocketChat#20461)
  Update Apps-Engine and permissions translations (RocketChat#20491)
  Regression: Add tests to new banners REST endpoints (RocketChat#20492)
  [IMPROVE] Autofocus on directory (RocketChat#20509)
  Update "Industry" setting (RocketChat#20510)
  ...
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