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

refactor: Improve sidebar search list hook #28328

Merged
merged 3 commits into from Mar 10, 2023

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Mar 8, 2023

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@ggazzo ggazzo requested a review from a team as a code owner March 8, 2023 15:52
@ggazzo ggazzo force-pushed the chore/sidebar-search-spotlight branch from 0ef5fd1 to 440a50f Compare March 8, 2023 17:19
@codecov
Copy link

codecov bot commented Mar 8, 2023

Codecov Report

Merging #28328 (440a50f) into develop (f0077cf) will increase coverage by 0.03%.
The diff coverage is 83.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28328      +/-   ##
===========================================
+ Coverage    44.86%   44.90%   +0.03%     
===========================================
  Files          771      771              
  Lines        14979    14977       -2     
  Branches      2092     2090       -2     
===========================================
+ Hits          6721     6726       +5     
+ Misses        7962     7956       -6     
+ Partials       296      295       -1     
Flag Coverage Δ
e2e 44.86% <83.33%> (+0.03%) ⬆️

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

@ggazzo ggazzo changed the title Chore: Improve search list hook refactor: Improve search list hook Mar 10, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Mar 10, 2023
@ggazzo ggazzo added this to the 6.1.0 milestone Mar 10, 2023
@ggazzo ggazzo changed the title refactor: Improve search list hook refactor: Improve sidebar search list hook Mar 10, 2023
@ggazzo ggazzo merged commit be179f1 into develop Mar 10, 2023
@ggazzo ggazzo deleted the chore/sidebar-search-spotlight branch March 10, 2023 18:11
gabriellsh added a commit that referenced this pull request Mar 13, 2023
…ketChat/Rocket.Chat into matrixSearchTestBranch

* 'feat/federation-public-room-search-2' of github.com:RocketChat/Rocket.Chat: (42 commits)
  refactor: Replace `_.isObject` with `isObject` (#28376)
  fix: small tweaks
  chore: Remove old css (#28140)
  fix: Thread list not scrolling to bottom when sending a message. (#28346)
  refactor: Federation refactor towards services + file names improvement + E2E tests fixes (#28326)
  i18n: Language update from LingoHub 🤖 on 2023-03-10Z (#28365)
  refactor: Improve sidebar search list hook (#28328)
  Chore: Change the PR title check to use conventional commits (#28358)
  [IMPROVE] Galician onboarding translation (#27908)
  [IMPROVE] Adding input type=password to AWS secrets fields (#28159)
  [FIX] Normal and multiline messages adding extra spaces upon copying/pasting (#28187)
  Chore: remove unused prop in register workspace modal (#28317)
  [FIX] App crashing when switching to Hindi (hi-In) language (#28285)
  [NEW] Registered Loom oEmbed (#28339)
  fix: create dm users before join the room
  tests: add unit tests + small tweaks
  fix: fix invite links button logic
  chore: fix lint
  Chore: Remove/replace some Meteor packages (#28189)
  Chore: Unused/dead code removal (#28155)
  ...
@casalsgh casalsgh modified the milestones: 6.1.0, 6.2.0 Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: QA skipped stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants