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

Regression: Apps-Engine - Convert streams to buffers on file upload #20523

Merged
merged 6 commits into from
Feb 1, 2021

Conversation

d-gubert
Copy link
Member

Proposed changes (including videos or screenshots)

This is an implementation to accommodate the changes in API for the IPreFileUpload hook in the Apps-Engine. Explanation on the reasoning for it is here RocketChat/Rocket.Chat.Apps-engine#376

Issue(s)

Steps to test or reproduce

Further comments

@d-gubert d-gubert marked this pull request as draft January 30, 2021 23:26
@d-gubert d-gubert added this to the 3.11.0 milestone Jan 31, 2021
@d-gubert d-gubert marked this pull request as ready for review January 31, 2021 19:01
@d-gubert d-gubert requested a review from rodrigok January 31, 2021 19:12
sampaiodiego
sampaiodiego previously approved these changes Jan 31, 2021
@sampaiodiego sampaiodiego merged commit d420c21 into develop Feb 1, 2021
@sampaiodiego sampaiodiego deleted the apps/file-upload branch February 1, 2021 00:18
@sampaiodiego sampaiodiego mentioned this pull request Feb 1, 2021
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