Skip to content

feat: can search exemplar IDs by user login, WEB-1020#628

Open
pleary wants to merge 1 commit into
mainfrom
web-1020-add-ability-to-search-remarks-by-username
Open

feat: can search exemplar IDs by user login, WEB-1020#628
pleary wants to merge 1 commit into
mainfrom
web-1020-add-ability-to-search-remarks-by-username

Conversation

@pleary
Copy link
Copy Markdown
Member

@pleary pleary commented May 28, 2026

No description provided.

@pleary pleary requested a review from caseyhofford May 28, 2026 20:16
Copy link
Copy Markdown
Contributor

@caseyhofford caseyhofford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, tested locally and everything works. Patrick has already added these mappings in production.

Comment thread schema/fixtures.js
"user": {
"id": 121,
"login": "user121",
"login_autocomplete": "user121"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to add the login_exact field to these two fixtures as well?

Suggested change
"login_autocomplete": "user121"
"login_autocomplete": "user121",
"login_exact": "user121"

}
}
},
"login": {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand how login and login_exact are being used, since we only match on login_autocomplete in the controller. Are these just different matchers that we provide access to for all strings that should be queryable for future use cases?

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.

2 participants