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

datacollector: find user by insensitive case name_abbreviation #1302

Conversation

PiTrem
Copy link
Member

@PiTrem PiTrem commented Apr 26, 2023

fallback to case insensitive result unless multiple users found

@PiTrem PiTrem linked an issue Apr 26, 2023 that may be closed by this pull request
app/models/user.rb Outdated Show resolved Hide resolved
app/models/user.rb Outdated Show resolved Hide resolved
app/models/user.rb Outdated Show resolved Hide resolved
spec/models/user_spec.rb Show resolved Hide resolved
@github-actions
Copy link

LCOV of commit f66ced9 during Continuous Integration #904

Summary coverage rate:
  lines......: 60.1% (11775 of 19608 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem force-pushed the 1301-datacollector-to-find-recipient-with-case-insensitive-name-abbreviation branch from f66ced9 to 18caf95 Compare April 26, 2023 10:31
@PiTrem PiTrem force-pushed the 1301-datacollector-to-find-recipient-with-case-insensitive-name-abbreviation branch from 18caf95 to 0eb8b34 Compare April 26, 2023 10:42
@github-actions
Copy link

LCOV of commit 18caf95 during Continuous Integration #905

Summary coverage rate:
  lines......: 60.1% (11775 of 19608 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link

LCOV of commit 0eb8b34 during Continuous Integration #906

Summary coverage rate:
  lines......: 60.0% (11772 of 19608 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem merged commit cb3934d into main Apr 26, 2023
2 checks passed
PiTrem added a commit that referenced this pull request Apr 28, 2023
return nil if any ambiguity (multiple users found)
dpatelev pushed a commit that referenced this pull request May 10, 2023
return nil if any ambiguity (multiple users found)
mekkyz pushed a commit that referenced this pull request Sep 21, 2023
return nil if any ambiguity (multiple users found)
baolanlequang pushed a commit that referenced this pull request Mar 5, 2024
return nil if any ambiguity (multiple users found)
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.

datacollector to find recipient with case insensitive name abbreviation
2 participants