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: Remove OS version for device management #27786

Merged
merged 5 commits into from
Jan 27, 2023
Merged

Conversation

LucianoPierdona
Copy link
Contributor

@LucianoPierdona LucianoPierdona commented Jan 18, 2023

Proposed changes (including videos or screenshots)

This PR removes the OS version from the device management page, this is because the user-agent is returning the wrong version.

Issue(s)

Steps to test or reproduce

  • Login into Rocket.Chat
  • Go to My Account -> Manage Devices
  • It shows the wrong version of your device

Further comments

TC-166

@codecov
Copy link

codecov bot commented Jan 18, 2023

Codecov Report

Merging #27786 (f10bcda) into develop (3cfa0b9) will increase coverage by 1.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27786      +/-   ##
===========================================
+ Coverage    41.95%   42.99%   +1.04%     
===========================================
  Files          847      820      -27     
  Lines        17758    17241     -517     
  Branches      2020     1945      -75     
===========================================
- Hits          7450     7413      -37     
+ Misses       10041     9562     -479     
+ Partials       267      266       -1     
Flag Coverage Δ
e2e 42.99% <ø> (+1.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

hugocostadev
hugocostadev previously approved these changes Jan 19, 2023
@LucianoPierdona LucianoPierdona added this to the 6.0.0 milestone Jan 20, 2023
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jan 23, 2023
@dionisio-bot dionisio-bot bot removed the stat: ready to merge PR tested and approved waiting for merge label Jan 23, 2023
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Jan 25, 2023
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Jan 26, 2023

This PR could not be merged because the GitHub API returned an internal server error. To enable Kodiak on this pull request please remove the kodiak:disabled label.

When the GitHub API returns an internal server error (HTTP status code 500), it is not safe for Kodiak to retry merging.

For more information please see https://kodiakhq.com/docs/troubleshooting#merge-errors

If you need help, you can open a GitHub issue, check the docs, or reach us privately at support@kodiakhq.com.

docs | dashboard | support

@ggazzo ggazzo merged commit 6ce320e into develop Jan 27, 2023
@ggazzo ggazzo deleted the fix/remove-os-version branch January 27, 2023 20:10
gabriellsh added a commit that referenced this pull request Feb 6, 2023
….Chat into matrixSearch

* 'feat/federation-feat-2' of github.com:RocketChat/Rocket.Chat: (31 commits)
  fix: removing file wrongly merged on update merge
  [NEW] Federation search public rooms in the Matrix network (#27434)
  Chore: Refactor other kinds of message - Phase 2 (#27927)
  Chore: improve some e2e selectors (#27936)
  [NEW] Warning Popup reinforcing Zapier deprecation on Integrations Admin settings (#27744)
  Chore: Notify Marketplace on App Install (#27925)
  i18n: Language update from LingoHub 🤖 on 2023-01-23Z (#27814)
  [FIX][ENTERPRISE] DDP streamer sending data to destroyed streams (#27929)
  Chore: Migrate cached collections to TypeScript (#27909)
  i18n: Language update from LingoHub 🤖 on 2023-01-09Z (#27705)
  [FIX] Reduce re-subscription on `notify-room/user-activity` stream event (#27911)
  [FIX] Unread Message count not displayed for new messages in Omni-Rooms (#27539)
  Chore: Improve service's licence check (#27872)
  [IMPROVE] Livechat Persian translation (#27825)
  Chore(deps): Bump thehanimo/pr-title-checker from 1.3.4 to 1.3.6 (#27884)
  Chore: use useQuery in place of deprecated useEndpointData (#27673)
  Chore: Update Some Composer Icons (#27862)
  Regression: Undefined default message context (#27870)
  Chore: Remove OS version for device management (#27786)
  [BREAK] Remove less theming (#27647)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Feb 17, 2023
Kishn0109 pushed a commit to Kishn0109/Rocket.Chat that referenced this pull request May 26, 2023
Kishn0109 pushed a commit to Kishn0109/Rocket.Chat that referenced this pull request May 26, 2023
Co-authored-by: Hugo Costa <hugocarreiracosta@gmail.com>
janainaCoelhoRocketchat pushed a commit that referenced this pull request Aug 18, 2023
Co-authored-by: Hugo Costa <hugocarreiracosta@gmail.com>
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.

4 participants