Skip to content

Conversation

@ikyuchukov
Copy link
Contributor

@ikyuchukov ikyuchukov commented Dec 7, 2020

Proposed changes

Sets the Global Search option to not be set by default.

Issue(s)

Currently Global Chat search is set to be used by default, this causes a lot of stress on servers with bigger chat histories and usually only bring not needed results as users usually search for commonly used words in their local chats. When an actual more refined search is used, global can be applied by checking the checkbox.

How to test or reproduce

Screenshots

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Changelog

Global chat search is not set by default now.

Further comments

@CLAassistant
Copy link

CLAassistant commented Dec 7, 2020

CLA assistant check
All committers have signed the CLA.

@wreiske
Copy link
Contributor

wreiske commented Dec 7, 2020

Almost 100% of the time I uncheck the global search. 👍

@shahariaazam
Copy link

By default no options should be selected by default. This issue is an important one specially for large number of histories.

@sampaiodiego sampaiodiego changed the title [FIX] [MINOR] Remove global search by default [IMPROVE] Don't use global search by default Jan 8, 2021
@sampaiodiego sampaiodiego merged commit 4144a36 into RocketChat:develop Jan 8, 2021
gabriellsh added a commit that referenced this pull request Jan 12, 2021
…info_page

* 'develop' of github.com:RocketChat/Rocket.Chat: (31 commits)
  [FIX] Normalize messages for users in endpoint chat.getStarredMessages (#19962)
  [FIX] minWidth in FileIcon to prevent layout to broke (#19942)
  [IMPROVE] Add extra SAML settings to update room subs and add private room subs (#19489)
  [IMPROVE] Rewrite User Dropdown and Kebab menu. (#20070)
  Language update from LingoHub 🤖 on 2021-01-11Z (#20146)
  [FIX] User registration updating wrong subscriptions (#20128)
  Rewrite Discussion Metric (#20117)
  [IMPROVE] Don't use global search by default (#19777)
  [FIX] Tabbar is opened (#20122)
  [FIX] Actions from User Info panel (#20073)
  Update password policy English translation (#20118)
  [FIX] Messages being updated when not required after user changes his profile (#20114)
  Custom fields not storing additional form values, fixing label and solving duplicated register. (#19953)
  Fix livechat room breaking after return (#20089)
  [FIX] Meteor errors not translating for toast messages (#19993)
  [FIX] Profile picture changing with username (#19992)
  [FIX] Search list filter (#19937)
  chore: Change return button (#20045)
  Rewrite : Message Thread metrics (#20051)
  [FIX] Omnichannel raw model importing meteor dependency (#20093)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Feb 1, 2021
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.

6 participants