Skip to content

Feat/privilege numbers #510

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Feb 6, 2025
Merged

Feat/privilege numbers #510

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
76b949f
Add privilege id
jusdino Jan 29, 2025
45f8778
Update libs, improve logging
jusdino Jan 29, 2025
6c56090
Check for unexpected license types
jusdino Jan 29, 2025
4edd9d7
Add invalid license type test
jusdino Jan 30, 2025
5a536f4
use `.get()` for attestations
jusdino Jan 30, 2025
8cae2e9
Persist existing privilege id
jusdino Jan 30, 2025
fb75e4b
Update api docs
jusdino Jan 30, 2025
9b6c1db
Revise logger_inject_kwargs
jusdino Feb 5, 2025
40b7424
Merge branch 'development' into feat/privilege-numbers
jusdino Feb 5, 2025
d2a1c1d
Revise test env vars
jusdino Feb 5, 2025
e824126
Merge branch 'development' into feat/privilege-numbers
jusdino Feb 5, 2025
bc98e94
Fix intermittent test failure
jusdino Feb 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion backend/compact-connect/bin/generate_mock_license_csv_upload_file.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ def get_mock_license(i: int, *, compact: str, jurisdiction: str = None) -> dict:
jurisdiction = faker.state_abbr().lower()
license_data = {
# |Zero padded 4 digit int|
'ssn': f'{(i//1_000_000) % 1000:03}-{(i//10_000) % 100:02}-{(i % 10_000):04}',
'ssn': f'{(i // 1_000_000) % 1000:03}-{(i // 10_000) % 100:02}-{(i % 10_000):04}',
# Some have NPI, some don't
'npi': str(randint(1_000_000_000, 9_999_999_999)) if choice([True, False]) else None,
# Some have License number, some don't
Expand Down
14 changes: 5 additions & 9 deletions backend/compact-connect/cdk.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,19 +29,15 @@
],
"license_types": {
"aslp": [
"audiologist",
"speech-language pathologist",
"speech and language pathologist"
{"name": "audiologist", "abbreviation": "aud"},
{"name": "speech-language pathologist", "abbreviation": "slp"}
],
"octp": [
"occupational therapist",
"occupational therapy assistant"
{"name": "occupational therapist", "abbreviation": "ot"},
{"name": "occupational therapy assistant", "abbreviation": "ota"}
],
"coun": [
"licensed professional counselor",
"licensed mental health counselor",
"licensed clinical mental health counselor",
"licensed professional clinical counselor"
{"name": "licensed professional counselor", "abbreviation": "lpc"}
]
},
"compacts": [
Expand Down
2 changes: 1 addition & 1 deletion backend/compact-connect/common_constructs/stack.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ def __init__(self, *args, standard_tags: StandardTags, **kwargs):
@cached_property
def license_types(self):
"""Flattened list of all license types across all compacts"""
return [typ for comp in self.node.get_context('license_types').values() for typ in comp]
return [typ['name'] for compact in self.node.get_context('license_types').values() for typ in compact]

@cached_property
def common_env_vars(self):
Expand Down
Loading