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 so that no expertise code in PLR record doesn't cause error #1744

Merged
merged 10 commits into from
Nov 10, 2021

Conversation

neophyte57
Copy link
Collaborator

@neophyte57 neophyte57 commented Nov 5, 2021

The easiest way to get some data into PlrProvider table is to run the console app in moh-prime\utilities\PlrIntakeUtility: dotnet run .\PRIME_Test_Data_PLR_IAT20210617_v2.0.xls intake.log

The PLR information panel should appear on the Enrollee Overview page if an Enrollee's Certification.LicenseNumber matches a PlrProvider.CollegeId.

Some test scenarios:

Scenario Expected
Enrollee does not match existing PLR record Overview of Enrollee displays properly, albeit without PLR panel
Enrollee matches PLR record that has no Expertises Overview of Enrollee displays OK, PLR panel displays "N/A" or equivalent for Expertise
Enrollee matches PLR record that has Expertises Expertises translated to human-readable text
Non-CPSID Licence Number, e.g. for Registered Midwife that matches PLR record Overview of Enrollee displays OK, PLR panel displays with PLR details
MOA Enrollee Overview of Enrollee displays properly, albeit without PLR panel
Enrollee has multiple certifications, some of which match a PLR record Overview of Enrollee displays with multiple PLR panels

@neophyte57 neophyte57 self-assigned this Nov 5, 2021
@neophyte57 neophyte57 added Backend Updates to the backend of the application Frontend Updates to the frontend of the application Ready For Review PR is ready for code review labels Nov 5, 2021
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 17:48 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 17:48 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 17:48 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 17:48 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 17:48 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 18:00 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 18:55 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 18:55 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 18:55 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 18:55 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 18:55 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 8, 2021 19:00 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 10, 2021 18:49 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 10, 2021 18:49 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 10, 2021 18:49 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 10, 2021 18:49 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 10, 2021 18:49 Inactive
@neophyte57 neophyte57 temporarily deployed to Development November 10, 2021 18:55 Inactive
@neophyte57 neophyte57 merged commit 006b9b9 into develop Nov 10, 2021
@neophyte57 neophyte57 deleted the fix/no-expertise-code-causes-error branch November 10, 2021 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend Updates to the backend of the application Frontend Updates to the frontend of the application Ready For Review PR is ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants