Frontend/cosmo search updates#1308
Merged
isabeleliassen merged 18 commits intocsg-org:mainfrom Apr 7, 2026
Merged
Conversation
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds cosmetology app-mode support across UI, store, forms, network, models, locales, mocks, and tests: introduces mode getters/dispatches, mode-conditional search fields (licenseNumber/dob) and columns, propagates new fields through search/license APIs, updates date validation and sorting. Changes
Sequence Diagram(s)sequenceDiagram
participant UI as rgba(79,129,189,0.5) User UI
participant Store as rgba(124,181,71,0.5) Vuex Store
participant Components as rgba(192,80,77,0.5) Vue Components
participant API as rgba(155,89,182,0.5) Network API
UI->>Store: dispatch setCurrentCompact(compact)
Store->>Store: dispatch setAppMode(AppModes.COSMETOLOGY|JCC)
Store->>Store: dispatch getCompactStatesRequest(compact.type)
Store-->>Components: getters isAppMode* update
UI->>Components: render mode-conditional fields (licenseNumber/dob or privilege fields)
UI->>API: submit search params (includes licenseNumber/dob when cosmetology)
API-->>Components: return search results
Components->>UI: render rows/columns per app mode
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
webroot/src/network/licenseApi/data.api.ts (1)
127-147:⚠️ Potential issue | 🟠 Major
licenseNumber-only searches are currently ignored.
hasSearchTermsdoes not includelicenseNumber, sorequestParams.query.licenseNumberis never set unless id/first/last name is also provided.Suggested fix
- const hasSearchTerms = Boolean(licenseeId || licenseeFirstName || licenseeLastName); + const hasSearchTerms = Boolean(licenseeId || licenseeFirstName || licenseeLastName || licenseNumber);🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/network/licenseApi/data.api.ts` around lines 127 - 147, The current gating variable hasSearchTerms excludes licenseNumber, so requestParams.query.licenseNumber is never set for license-number-only searches; update the logic in data.api.ts by either adding licenseNumber to the hasSearchTerms expression (e.g., include licenseNumber in the Boolean(...) that defines hasSearchTerms) or move the licenseNumber assignment (requestParams.query.licenseNumber) out of the hasSearchTerms conditional so that licenseNumber is applied independently; ensure you reference and modify the hasSearchTerms variable and the requestParams.query.licenseNumber assignment accordingly.
🧹 Nitpick comments (3)
webroot/src/store/user/user.spec.ts (1)
1086-1090: Strengthen the compact-states dispatch assertion with payload checks.Both tests verify the second dispatched action name, but not the compact type argument. Add payload checks so regressions in downstream state-fetch routing are caught.
Suggested test assertion tightening
- expect([dispatch.secondCall.args[0]]).to.matchPattern(['getCompactStatesRequest']); + expect(dispatch.secondCall.args).to.matchPattern(['getCompactStatesRequest', CompactType.ASLP]); ... - expect([dispatch.secondCall.args[0]]).to.matchPattern(['getCompactStatesRequest']); + expect(dispatch.secondCall.args).to.matchPattern(['getCompactStatesRequest', CompactType.COSMETOLOGY]);Also applies to: 1100-1103
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/store/user/user.spec.ts` around lines 1086 - 1090, The test currently only asserts the second dispatched action name; update the assertion to also verify the payload/argument passed to getCompactStatesRequest (e.g., inspect dispatch.secondCall.args[1] or the appropriate payload position) equals the expected compact type so downstream routing won't regress; change the assertion in the block referencing dispatch and getCompactStatesRequest (and mirror the same tightening for the similar assertions around lines with the other test) to assert both action name and payload value alongside existing AppModes.JCC checks.webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.ts (2)
376-389: Consider logging unhandled app modes in the default case.The switch statement cleanly separates mode-specific search properties. However, the empty
defaultcase silently falls through. If a newAppModesvalue is added in the future without updating this switch, it would use only common search props without any indication.Consider adding a console warning for development or using TypeScript's exhaustive check pattern.
💡 Optional: Add exhaustive check or warning
case AppModes.COSMETOLOGY: allowedSearchProps.push('licenseNumber'); break; -default: - break; +default: { + // Exhaustive check - TypeScript will error if a new AppModes value is added + const _exhaustiveCheck: never = this.appMode; + console.warn(`Unhandled app mode: ${_exhaustiveCheck}`); + break; +} }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.ts` around lines 376 - 389, The switch on this.appMode (inside LicenseeSearch where allowedSearchProps is built) currently has an empty default and will silently ignore any new AppModes values; add a dev-only warning or an exhaustive check in the default branch to surface unhandled app modes (e.g., console.warn or throw in non-production), or implement a TypeScript exhaustive check using a helper that accepts the unreachable value, referencing AppModes, allowedSearchProps, and the switch block so future additions to AppModes are detected and logged.
257-265: Consider adding a placeholder for consistency.The
licenseNumberFormInput has an empty placeholder while similar fields likefirstNameandlastNameuse computed placeholders. If a placeholder would help users, consider adding one.💡 Optional: Add a placeholder
licenseNumber: new FormInput({ id: 'license-number', name: 'license-number', label: computed(() => this.$t('licensing.licenseNumber')), - placeholder: '', + placeholder: computed(() => this.$t('licensing.searchPlaceholderLicenseNumber')), validation: Joi.string().min(0).max(100).messages(this.joiMessages.string), value: this.searchParams.licenseNumber || '', enforceMax: true, }),🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.ts` around lines 257 - 265, Add a computed placeholder to the licenseNumber FormInput (the new FormInput instance assigned to licenseNumber) to match other fields: replace the empty placeholder with a computed translation like computed(() => this.$t('licensing.licenseNumberPlaceholder')) or an appropriate i18n key so the field shows a consistent placeholder text; update the i18n strings if needed to include that key.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@webroot/src/models/License/License.model.ts`:
- Around line 33-34: Rename the misspelled enum key COSMOTOLOGIST to
COSMETOLOGIST in the License enum (License.model.ts) so the key matches its
value 'cosmetologist'; update the enum declaration (replace COSMOTOLOGIST =
'cosmetologist' with COSMETOLOGIST = 'cosmetologist') and run a quick global
search/replace for COSMOTOLOGIST to ensure no references remain, adjusting any
usages to the new COSMETOLOGIST identifier.
In `@webroot/src/network/mocks/mock.data.ts`:
- Around line 188-225: In the mock permissions objects for regions ne, oh, nv,
and ma inside the actions blocks, replace the incorrect key readSsn with the
canonical readSSN to match the rest of the mock schema; update each actions
object (e.g., the actions object nested under the ne, oh, nv, ma entries) so the
permission key is readSSN and run a quick search to ensure no other occurrences
of readSsn remain.
---
Outside diff comments:
In `@webroot/src/network/licenseApi/data.api.ts`:
- Around line 127-147: The current gating variable hasSearchTerms excludes
licenseNumber, so requestParams.query.licenseNumber is never set for
license-number-only searches; update the logic in data.api.ts by either adding
licenseNumber to the hasSearchTerms expression (e.g., include licenseNumber in
the Boolean(...) that defines hasSearchTerms) or move the licenseNumber
assignment (requestParams.query.licenseNumber) out of the hasSearchTerms
conditional so that licenseNumber is applied independently; ensure you reference
and modify the hasSearchTerms variable and the requestParams.query.licenseNumber
assignment accordingly.
---
Nitpick comments:
In `@webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.ts`:
- Around line 376-389: The switch on this.appMode (inside LicenseeSearch where
allowedSearchProps is built) currently has an empty default and will silently
ignore any new AppModes values; add a dev-only warning or an exhaustive check in
the default branch to surface unhandled app modes (e.g., console.warn or throw
in non-production), or implement a TypeScript exhaustive check using a helper
that accepts the unreachable value, referencing AppModes, allowedSearchProps,
and the switch block so future additions to AppModes are detected and logged.
- Around line 257-265: Add a computed placeholder to the licenseNumber FormInput
(the new FormInput instance assigned to licenseNumber) to match other fields:
replace the empty placeholder with a computed translation like computed(() =>
this.$t('licensing.licenseNumberPlaceholder')) or an appropriate i18n key so the
field shows a consistent placeholder text; update the i18n strings if needed to
include that key.
In `@webroot/src/store/user/user.spec.ts`:
- Around line 1086-1090: The test currently only asserts the second dispatched
action name; update the assertion to also verify the payload/argument passed to
getCompactStatesRequest (e.g., inspect dispatch.secondCall.args[1] or the
appropriate payload position) equals the expected compact type so downstream
routing won't regress; change the assertion in the block referencing dispatch
and getCompactStatesRequest (and mirror the same tightening for the similar
assertions around lines with the other test) to assert both action name and
payload value alongside existing AppModes.JCC checks.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (27)
webroot/.eslintrc.jswebroot/src/components/CompactSelector/CompactSelector.tswebroot/src/components/Licensee/LicenseeList/LicenseeList.tswebroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.tswebroot/src/components/Licensee/LicenseeRow/LicenseeRow.tswebroot/src/components/Licensee/LicenseeRow/LicenseeRow.vuewebroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.tswebroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.vuewebroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.tswebroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.vuewebroot/src/components/PrivilegeCard/PrivilegeCard.tswebroot/src/components/PrivilegeCard/PrivilegeCard.vuewebroot/src/locales/en.jsonwebroot/src/locales/es.jsonwebroot/src/models/License/License.model.tswebroot/src/models/Licensee/Licensee.model.tswebroot/src/network/licenseApi/data.api.tswebroot/src/network/mocks/mock.data.tswebroot/src/network/searchApi/data.api.tswebroot/src/pages/LicensingDetail/LicensingDetail.lesswebroot/src/pages/LicensingDetail/LicensingDetail.tswebroot/src/pages/LicensingDetail/LicensingDetail.vuewebroot/src/store/global/global.getters.tswebroot/src/store/license/license.mutations.tswebroot/src/store/license/license.spec.tswebroot/src/store/user/user.actions.tswebroot/src/store/user/user.spec.ts
4 tasks
Contributor
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (1)
webroot/src/components/Licensee/LicenseeList/LicenseeList.ts (1)
395-397: GuardlicenseNumberserialization by app mode.For defensive correctness, only attach
licenseNumberwhen cosmetology mode is active.♻️ Suggested hardening
- if (searchParams?.licenseNumber) { + if (this.isAppModeCosmetology && searchParams?.licenseNumber) { requestConfig.licenseNumber = searchParams.licenseNumber; }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/components/Licensee/LicenseeList/LicenseeList.ts` around lines 395 - 397, Only set requestConfig.licenseNumber from searchParams?.licenseNumber when the app is running in cosmetology mode: guard the assignment by checking the app mode (e.g., call the existing isCosmetologyMode() helper or compare appMode === 'cosmetology') before assigning requestConfig.licenseNumber; leave it unset otherwise. Update the block around searchParams?.licenseNumber and requestConfig to perform that conditional check so license numbers are only serialized for cosmetology mode.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.ts`:
- Around line 299-301: Only set requestConfig.licenseNumber when the app is in
cosmetology mode: wrap the existing assignment that reads
searchParams.licenseNumber and assigns requestConfig.licenseNumber with an
additional condition that verifies cosmetology mode (e.g., check the relevant
mode flag/enum used in this component such as isCosmetologyMode or appMode ===
'cosmetology'); change the current block that uses searchParams.licenseNumber so
it only runs if both searchParams.licenseNumber is present and the
cosmetology-mode check passes, leaving all other requestConfig fields unchanged.
- Around line 127-135: The assembly of the combined search display in
searchDisplayAll should filter out empty/blank parts before joining to avoid
malformed separators; instead of building the array then joining and running
regex cleanup, filter the array elements (this.searchDisplayCompact,
this.searchDisplayFullName, this.searchDisplayState,
this.searchDisplayLicenseNumber) for truthy/non-blank values and then join them
with ', ' to produce the final string (remove the current joined variable +
regex replaces and return the filtered-then-joined result).
In
`@webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts`:
- Around line 249-263: The resetForm() method currently clears
this.formData.compact.value only when enableCompactSelect is true but does not
clear the store-backed compact state; update resetForm() (within
LicenseeSearchLegacy.resetForm) to also clear the compact state in the store
whenever you reset the form (respecting enableCompactSelect), e.g., invoke the
existing action/mutation or method that sets the store compact value (the same
mechanism used elsewhere to set compact state), and keep the existing resets for
firstName, lastName, state, licenseNumber and the calls to
updateFormSubmitSuccess and updateFormSubmitError so form submit state is fully
cleared.
In
`@webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.vue`:
- Around line 23-28: Replace the anchor used for the clear action with a native
button to restore keyboard accessibility: where the template conditionally
renders the anchor (v-if="isMockPopulateEnabled") that binds `@click` and
`@keyup.enter` to resetForm(), change it to a <button type="button"> that keeps
the same v-if, `@click`="resetForm()" and remove the redundant `@keyup.enter`
handler (buttons handle Enter/Space natively); ensure the CSS class "clear-form
search-input" is preserved and adjust any styling selectors if they relied on an
<a> element.
---
Nitpick comments:
In `@webroot/src/components/Licensee/LicenseeList/LicenseeList.ts`:
- Around line 395-397: Only set requestConfig.licenseNumber from
searchParams?.licenseNumber when the app is running in cosmetology mode: guard
the assignment by checking the app mode (e.g., call the existing
isCosmetologyMode() helper or compare appMode === 'cosmetology') before
assigning requestConfig.licenseNumber; leave it unset otherwise. Update the
block around searchParams?.licenseNumber and requestConfig to perform that
conditional check so license numbers are only serialized for cosmetology mode.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: b92c7b95-1df5-4cf8-851d-e4cb1af6a234
📒 Files selected for processing (5)
webroot/src/components/Licensee/LicenseeList/LicenseeList.tswebroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.tswebroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.lesswebroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.tswebroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.vue
webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.ts
Outdated
Show resolved
Hide resolved
webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.ts
Outdated
Show resolved
Hide resolved
webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts
Outdated
Show resolved
Hide resolved
webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.vue
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts (1)
249-264:⚠️ Potential issue | 🟡 MinorReset should also clear field-level validation state.
resetForm()clears values and submit-level flags, but it does not explicitly reset/recompute per-input touched/error state. Add a non-touched validation pass (or equivalent field reset) to avoid stale inline errors after clear.🧹 Suggested adjustment
async resetForm(): Promise<void> { if (this.enableCompactSelect) { this.formData.compact.value = ''; await this.$store.dispatch('user/setCurrentCompact', null); } this.formData.firstName.value = ''; this.formData.lastName.value = ''; this.formData.state.value = ''; this.formData.licenseNumber.value = ''; + this.validateAll({ asTouched: false }); this.isFormLoading = false; this.isFormSuccessful = false; this.isFormError = false; this.updateFormSubmitSuccess(''); this.updateFormSubmitError(''); }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts` around lines 249 - 264, resetForm currently clears values and submit flags but leaves per-field validation state intact; update resetForm (in LicenseeSearchLegacy.resetForm) to also clear field-level validation by either calling the form validation reset API (e.g., this.$v && this.$v.$reset()) or explicitly resetting each field's validation props: for compact, firstName, lastName, state, licenseNumber set touched/dirty flags to false and clear any error messages (e.g., field.touched = false; field.error = ''), then optionally re-run a non-touched validation pass if your app relies on computed validation state.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts`:
- Around line 219-221: handleSubmit() now emits licenseNumber for cosmetology
but the request mapper omits licenseNumber when other name/id fields are blank;
update the mapper that builds the outgoing request (the hasSearchTerms branch /
mapping logic) to explicitly check for and include query.licenseNumber (or move
the licenseNumber mapping out of the hasSearchTerms-only block) so that when
AppModes.COSMETOLOGY submits only licenseNumber it is serialized into the
request; adjust the conditional that computes hasSearchTerms or the mapping
function so licenseNumber is honored independently of firstName/lastName/id.
---
Duplicate comments:
In
`@webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts`:
- Around line 249-264: resetForm currently clears values and submit flags but
leaves per-field validation state intact; update resetForm (in
LicenseeSearchLegacy.resetForm) to also clear field-level validation by either
calling the form validation reset API (e.g., this.$v && this.$v.$reset()) or
explicitly resetting each field's validation props: for compact, firstName,
lastName, state, licenseNumber set touched/dirty flags to false and clear any
error messages (e.g., field.touched = false; field.error = ''), then optionally
re-run a non-touched validation pass if your app relies on computed validation
state.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 649aa13f-d463-428c-989d-30b28c18bd41
📒 Files selected for processing (2)
webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.tswebroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts
webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
webroot/src/network/searchApi/data.api.ts (1)
26-35:⚠️ Potential issue | 🔴 CriticalAdd
dateOfBirthto the OpenSearch document structure for cosmetology license searches.The frontend queries
licenseNumber(root level) andlicenses.dateOfBirth, but the backend'sLicenseGeneralResponseSchemaused ingenerate_provider_opensearch_document()includeslicenseNumberon line 168 but omitsdateOfBirthentirely.dateOfBirthonly exists inLicenseReadPrivateResponseSchema, which is not used for search documents. Without this field indexed, searches by date of birth will silently return no results.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/network/searchApi/data.api.ts` around lines 26 - 35, The OpenSearch document schema and generator are missing the licenses.dateOfBirth field required for cosmetology license searches: update LicenseGeneralResponseSchema to include dateOfBirth on the license object (same type/format used in LicenseReadPrivateResponseSchema) and modify generate_provider_opensearch_document() to populate licenses[].dateOfBirth from the source license data so licenses.dateOfBirth is indexed (ensure the field uses the correct string/ISO date format expected by the frontend query).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@webroot/src/network/searchApi/data.api.ts`:
- Around line 26-35: The OpenSearch document schema and generator are missing
the licenses.dateOfBirth field required for cosmetology license searches: update
LicenseGeneralResponseSchema to include dateOfBirth on the license object (same
type/format used in LicenseReadPrivateResponseSchema) and modify
generate_provider_opensearch_document() to populate licenses[].dateOfBirth from
the source license data so licenses.dateOfBirth is indexed (ensure the field
uses the correct string/ISO date format expected by the frontend query).
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 9646b276-3894-4639-a88e-383dfe481998
📒 Files selected for processing (6)
webroot/src/components/Licensee/LicenseeList/LicenseeList.tswebroot/src/components/Licensee/LicenseeRow/LicenseeRow.vuewebroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.tswebroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.vuewebroot/src/network/licenseApi/data.api.tswebroot/src/network/searchApi/data.api.ts
✅ Files skipped from review due to trivial changes (1)
- webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.vue
🚧 Files skipped from review as they are similar to previous changes (3)
- webroot/src/components/Licensee/LicenseeRow/LicenseeRow.vue
- webroot/src/network/licenseApi/data.api.ts
- webroot/src/components/Licensee/LicenseeList/LicenseeList.ts
rmolinares
reviewed
Apr 3, 2026
rmolinares
approved these changes
Apr 3, 2026
Collaborator
Author
|
@jlkravitz This is ready for your review. |
jlkravitz
requested changes
Apr 7, 2026
Collaborator
jlkravitz
left a comment
jlkravitz
left a comment
There was a problem hiding this comment.
few comments/questions!
webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts
Show resolved
Hide resolved
webroot/src/components/Licensee/LicenseeSearchLegacy/LicenseeSearchLegacy.ts
Show resolved
Hide resolved
Contributor
There was a problem hiding this comment.
Actionable comments posted: 5
🧹 Nitpick comments (1)
webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.vue (1)
26-31: Good accessibility improvement adding Enter key support.The
@keyup.enterhandler makes theCloseXelement properly keyboard-accessible alongside its click handler andtabindex="0".For complete button-like behavior, consider also adding
@keyup.spacesince users expect both Enter and Space to activate button-like controls:♻️ Optional enhancement
<CloseX class="search-terms-reset" `@click`="resetSearch()" `@keyup.enter`="resetSearch()" + `@keyup.space.prevent`="resetSearch()" tabindex="0" />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.vue` around lines 26 - 31, The CloseX element currently handles click and Enter key via `@click`="resetSearch()" and `@keyup.enter`="resetSearch()" but lacks Space-key activation; add an additional keyboard handler to call resetSearch() on Space (e.g., `@keyup.space`="resetSearch()") so CloseX behaves like a button for keyboard users, and ensure the handlers reference the existing resetSearch method on the component.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@webroot/src/components/Licensee/LicenseeList/LicenseeList.ts`:
- Around line 114-118: The getter searchDisplayDob is parsing the user-entered
dob using the wrong format (it currently uses serverDateFormat); update the
moment parsing to use the input format 'MM/DD/YYYY' (i.e., moment(dob,
'MM/DD/YYYY')) and keep the existing .format(displayDateFormat) output; ensure
this change is applied inside the searchDisplayDob getter so the displayed DOB
renders correctly for user-entered dates.
In `@webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.ts`:
- Around line 446-449: The mock populate for cosmetology mode sets
this.formData.dob.value using moment(...).format('YYYY-MM-DD') which doesn't
match the form's expected MM/DD/YYYY validation; update the cosmetology branch
(the else if (this.isAppModeCosmetology) block) to format the date as MM/DD/YYYY
(e.g., use moment('1970-01-01').format('MM/DD/YYYY') or an equivalent) so
this.formData.dob.value matches the form input validation.
In `@webroot/src/components/PrivilegeCard/PrivilegeCard.vue`:
- Around line 441-444: The PrivilegeCard.vue shows privilegeId conditionally in
the form using isAppModeJcc but renders privilegeId unconditionally in the "add
investigation success" and "end investigation success" blocks; update those two
success-state templates to wrap the static-value (or the entire static-input)
that renders privilegeId with v-if="isAppModeJcc" (the same conditional used
earlier) so the privilegeId is shown only in JCC app mode; search for
privilegeId in the success-state markup near the add investigation success and
end investigation success sections and apply the same conditional.
- Around line 206-209: The success confirmation view still shows the privilegeId
unconditionally while the form hides it when isAppModeJcc is false; update the
success confirmation block that renders the static-value for privilegeId to use
the same v-if="isAppModeJcc" conditional (i.e., wrap or add v-if to the same
element/class that displays privilegeId in the success/confirmation section) so
the privilegeId visibility matches the form state; check the PrivilegeCard
component and update the confirmation rendering that references privilegeId
accordingly.
In `@webroot/src/network/searchApi/data.api.ts`:
- Around line 202-213: The DOB term query is using the user-provided MM/DD/YYYY
string (dob) which won't match the index's YYYY-MM-DD format; convert dob from
MM/DD/YYYY to ISO YYYY-MM-DD before pushing the nested term query (i.e.,
transform the dob variable in the block that builds the nested { path:
'licenses', query: { term: { 'licenses.dateOfBirth': dob } } } ). Implement the
conversion either inline in the same function in data.api.ts or call a small
helper (e.g., formatDobToIso or parseDob) that splits MM/DD/YYYY and reorders to
YYYY-MM-DD, and only push the condition if the conversion succeeds (otherwise
skip or handle invalid format).
---
Nitpick comments:
In `@webroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.vue`:
- Around line 26-31: The CloseX element currently handles click and Enter key
via `@click`="resetSearch()" and `@keyup.enter`="resetSearch()" but lacks Space-key
activation; add an additional keyboard handler to call resetSearch() on Space
(e.g., `@keyup.space`="resetSearch()") so CloseX behaves like a button for
keyboard users, and ensure the handlers reference the existing resetSearch
method on the component.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 6fc55ba8-63a3-4936-af6f-f490e07f70d5
📒 Files selected for processing (13)
webroot/src/components/Forms/InputDate/InputDate.tswebroot/src/components/Forms/_mixins/form.mixin.tswebroot/src/components/Licensee/LicenseeList/LicenseeList.tswebroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.lesswebroot/src/components/Licensee/LicenseeListLegacy/LicenseeListLegacy.vuewebroot/src/components/Licensee/LicenseeRow/LicenseeRow.vuewebroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.tswebroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.vuewebroot/src/components/PrivilegeCard/PrivilegeCard.vuewebroot/src/locales/en.jsonwebroot/src/locales/es.jsonwebroot/src/network/licenseApi/data.api.tswebroot/src/network/searchApi/data.api.ts
✅ Files skipped from review due to trivial changes (1)
- webroot/src/locales/en.json
🚧 Files skipped from review as they are similar to previous changes (4)
- webroot/src/network/licenseApi/data.api.ts
- webroot/src/components/Licensee/LicenseeRow/LicenseeRow.vue
- webroot/src/locales/es.json
- webroot/src/components/Licensee/LicenseeSearch/LicenseeSearch.vue
Collaborator
Author
|
@jlkravitz This is ready for re-review. |
jlkravitz
approved these changes
Apr 7, 2026
Collaborator
jlkravitz
left a comment
jlkravitz
left a comment
There was a problem hiding this comment.
@isabeleliassen good to merge this one!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requirements List
Description List
eslintrcfor usingv-forkeys in<template>elementsTODO
Testing List
yarn test:unit:allshould run without errors or warningsyarn serveshould run without errors or warningsyarn buildshould run without errors or warningsCloses #1268
Closes #1269
Closes #1280
Summary by CodeRabbit
New Features
Improvements