-
Notifications
You must be signed in to change notification settings - Fork 1
[GPCAPIM-275]: Harmonise the Controller, Flask entry point and Provider System modules #75
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
base: main
Are you sure you want to change the base?
Changes from all commits
cc6d5c3
4ebe273
742f2fd
41437ac
dd8c318
4f1534d
3580a2b
22adc40
4b9dbec
80a90ed
72af28c
a41f241
701038b
5e55b85
5d924a8
e5fecaa
8354daf
36d9812
ddd5b94
750e4d4
7aa9fa2
5af0f0a
366f6d4
4169586
be94937
0c3c8cd
30fdbaa
6dc0873
ebcb6e2
39096cb
1879d55
553e8a6
fef17b3
847a65a
5a1c445
f4b026b
33f0e81
332c64c
8af8423
a2117e3
d85c426
6ed656f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| { | ||
| // Use IntelliSense to learn about possible attributes. | ||
| // Hover to view descriptions of existing attributes. | ||
| // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387 | ||
| "version": "0.2.0", | ||
| "configurations": [ | ||
|
|
||
| { | ||
| "name": "Run Gateway API", | ||
| "type": "debugpy", | ||
| "request": "launch", | ||
| "program": "${workspaceFolder}/gateway-api/src/gateway_api/app.py", | ||
| "envFile": "${workspaceFolder}/.env", | ||
| "jinja": true, | ||
| "console": "integratedTerminal" | ||
| } | ||
| ] | ||
| } | ||
|
Comment on lines
1
to
18
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does it make sense to add this? As I understand it we're about to be asked to drop Flask.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is quite a generic configuration (the |
||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| """FHIR GeneralPractitioner type.""" | ||
|
|
||
| from typing import TypedDict | ||
|
|
||
| from fhir.period import Period | ||
|
|
||
|
|
||
| class GeneralPractitionerIdentifier(TypedDict): | ||
| """Identifier for GeneralPractitioner with optional period.""" | ||
|
|
||
| system: str | ||
| value: str | ||
| period: Period | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should Period be NotRequired if it's optional?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. AFAIK, |
||
|
|
||
|
|
||
| class GeneralPractitioner(TypedDict): | ||
| """FHIR GeneralPractitioner reference.""" | ||
|
|
||
| id: str | ||
| type: str | ||
| identifier: GeneralPractitionerIdentifier | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're already using
valeto provide some spell checking within the repository. Does this extension integrate with this?