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

[BREAK] Remove Google Vision features #23160

Merged
merged 2 commits into from
Sep 10, 2021
Merged

Conversation

sampaiodiego
Copy link
Member

Proposed changes (including videos or screenshots)

Google Vision features like "block adult images" or label detection were not being maintained and totally broken. So we decided to remove its feature and maybe in the future release the same features as an app.

Issue(s)

Closes #7926
Closes #10168
Closes #13166

Steps to test or reproduce

Further comments

@sampaiodiego sampaiodiego requested review from a team September 9, 2021 19:10
@lgtm-com
Copy link

lgtm-com bot commented Sep 9, 2021

This pull request fixes 1 alert when merging af87c54 into f710bb5 - view on LGTM.com

fixed alerts:

  • 1 for Identical operands

@ggazzo ggazzo merged commit c292848 into develop Sep 10, 2021
@ggazzo ggazzo deleted the remove-google-vision-features branch September 10, 2021 12:46
gabriellsh added a commit that referenced this pull request Sep 16, 2021
…to feat/seats-cap-banners

* 'feat/seats-cap' of github.com:RocketChat/Rocket.Chat:
  [NEW] Seats Cap: Request seats link (#23151)
  Regression: View Logs administration page crashing (#23205)
  Chore: Move client helpers (#23178)
  Chore: Change Ubuntu version to 20.04 on all GitHub Actions (#23200)
  Regression: Properly trickle-down state from UsersPage to UsersTable (#23196)
  [IMPROVE] Add missing pt-BR translations, fix typos and unify language (#23176)
  [FIX] User list not being updated after creation/deletion of user (#23032)
  Chore: Upgrade limax (#23187)
  [FIX] "Parent channel or group" search in discussions' creation throws "Unexpected end of JSON input" error (#23076)
  [BREAK] Stop sending audio notifications via stream (#23108)
  Language update from LingoHub 🤖 (#23184)
  [FIX] Remove margin from quote inside quote (#21779)
  [BREAK] Remove Google Vision features (#23160)
  Bump @storybook/react from 6.3.6 to 6.3.8 (#23165)
  Bump jsrsasign from 10.3.0 to 10.4.0 (#23163)
  [BREAK][ENTERPRISE] "Download CSV" button doesn't work in the Engagement Dashboard's Active Users section (#23013)
  [BREAK][ENTERPRISE] CSV file downloaded in the Engagement Dashboard's New Users section contains undefined data (#23014)
  Bump supertest from 6.1.3 to 6.1.6 (#23139)
@sampaiodiego sampaiodiego mentioned this pull request Oct 1, 2021
@rsgodinez
Copy link

Google Vision still present in documentation https://docs.rocket.chat/guides/administration/settings/file-upload#google-vision - it actually was confusing for me :)

@Rodriq
Copy link
Contributor

Rodriq commented Mar 30, 2022

Google Vision still present in documentation https://docs.rocket.chat/guides/administration/settings/file-upload#google-vision - it actually was confusing for me :)

The docs has been updated https://docs.rocket.chat/guides/administration/settings/file-upload

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants