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

fix: in forward search field, user cannot be found by name (Full Name) #29663

Merged
merged 11 commits into from
Oct 3, 2023

Conversation

Rottenblasters
Copy link
Contributor

@Rottenblasters Rottenblasters commented Jun 28, 2023

Proposed changes (including videos or screenshots)

Untitled.mp4

You can now search with both username and full name / name in share message search field.

Previously the query to find users used only the lowerCaseName field which used to store the lowercase version of the username. Now I have changed in to query either by lowerCaseName or lowerCaseFName (which stores the lowercase version of the full name of the user)

Issue(s)

Closes #29328
Closes #29632

Steps to test or reproduce

Further comments

SUP-327

@Rottenblasters Rottenblasters requested a review from a team as a code owner June 28, 2023 10:35
@changeset-bot
Copy link

changeset-bot bot commented Jun 28, 2023

🦋 Changeset detected

Latest commit: 165f646

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Rottenblasters Rottenblasters changed the title Fix: Issue#29328 (in forward search field, user cannot be found by name (Full Name)) fix: Issue#29328 (in forward search field, user cannot be found by name (Full Name)) Jun 28, 2023
@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #29663 (61591eb) into develop (aa30c85) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop   #29663   +/-   ##
========================================
  Coverage    50.93%   50.93%           
========================================
  Files          809      809           
  Lines        15016    15016           
  Branches      2724     2724           
========================================
  Hits          7649     7649           
  Misses        6958     6958           
  Partials       409      409           
Flag Coverage Δ
unit 62.93% <ø> (ø)

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

@Rottenblasters Rottenblasters changed the title fix: Issue#29328 (in forward search field, user cannot be found by name (Full Name)) fix: in forward search field, user cannot be found by name (Full Name) Jun 28, 2023
@MrPoint1400
Copy link

+1

@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Sep 26, 2023
@dionisio-bot dionisio-bot bot removed the stat: ready to merge PR tested and approved waiting for merge label Sep 26, 2023
@yash-rajpal yash-rajpal added the stat: ready to merge PR tested and approved waiting for merge label Sep 28, 2023
@dionisio-bot dionisio-bot bot removed the stat: ready to merge PR tested and approved waiting for merge label Oct 2, 2023
@ggazzo ggazzo merged commit b14e159 into RocketChat:develop Oct 3, 2023
26 checks passed
gabriellsh added a commit that referenced this pull request Oct 3, 2023
…e/translationCdn

* 'develop' of github.com:RocketChat/Rocket.Chat: (117 commits)
  chore: do not focus messagebox on mobile devices (#30553)
  chore: add the new endpoints to sync with cloud (#30546)
  fix: in forward search field, user cannot be found by name (Full Name) (#29663)
  chore: set license public key v3 with v2 (#30548)
  chore: adjust callbacks return type (#30547)
  chore: Delete Trigger Endpoint (#30533)
  test: adding missing verifications on message-actions (#30531)
  refactor: Oembed backend (#30228)
  test: refactor API E2E tests for groups endpoints (#30523)
  test: adding missing verifications on threads (#30528)
  chore: refactor cloud sync (#30401)
  refactor: Omni logs (#30322)
  refactor: Livechat Header -> ts (#30255)
  chore: Deprecate un-used meteor method for omnichannel analytics (#30421)
  feat: add supported versions + minimum clients versions to the info endpoint (#30178)
  feat: Auto-enable autotranslate (#30370)
  chore: Prevent call license and registration status endpoints when not enough permission (#30336)
  ci: Security/GitHub jira integration (#29776)
  feat: License V3 (#30287)
  chore: move Omnichannel toolbox section from sidebar room list (#30502)
  ...
gabriellsh added a commit that referenced this pull request Oct 6, 2023
…t into improve/ui-caching

* 'improve/ui-caching' of github.com:RocketChat/Rocket.Chat: (40 commits)
  Update .changeset/wicked-humans-hang.md
  feat: Save visitor's activity on agent's interaction (#30222)
  chore: update mentions upsell modal content (#30503)
  fix: font-disabled color (#30569)
  chore: anchor text-decoration rules (#30485)
  chore: `AccessibilityPage` hide Enterprise tag when is EE (#30486)
  chore: Replace `Field.[SUBCOMPONENT]` in favor of named imports (#30501)
  fix: Check for room scoped permissions for creating discussions (#30497)
  ci: fix Docker image build for production (#30562)
  chore: Fix triggers flaky tests (#30530)
  chore: get translations from CDN (#30331)
  feat: new `licenses.info` endpoint (#30473)
  ci: run tests from forks (#30556)
  regression: unmarked dangling promise on license validation (#30557)
  regression: fix initializing startup order (#30555)
  feat: push notification statistics (#30269)
  chore: do not focus messagebox on mobile devices (#30553)
  chore: add the new endpoints to sync with cloud (#30546)
  fix: in forward search field, user cannot be found by name (Full Name) (#29663)
  chore: set license public key v3 with v2 (#30548)
  ...
@casalsgh casalsgh added this to the 6.5.0 milestone Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
8 participants