-
Notifications
You must be signed in to change notification settings - Fork 333
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
Move "public" resources into R4 Project #2591
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
Discussed - test in staging. @mattwiller @codyebberson |
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.
Nice, this looks good.
I'll deploy to staging when I get a chance. Assuming that looks good, then this is good-to-go 👍
@@ -113,12 +113,14 @@ function getResourceTypesByExportLevel(exportLevel: string): ResourceType[] { | |||
return getResourceTypes(); | |||
} | |||
|
|||
const unexportedResourceTypes = ['CodeSystem', 'SearchParameter', 'StructureDefinition', 'ValueSet']; | |||
|
|||
function canBeExported(resourceType: string): boolean { | |||
if (resourceType === 'BulkDataExport') { |
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.
Now that we're introducing unexportedResourceTypes
, should we move the BulkDataExport
into that?
Then this method becomes return !protectedResourcesTypes.includes(resourceType) && !unexportedResourceTypes.includes(resourceType)
const updateQuery = `UPDATE "__TABLE__" SET | ||
"projectId" = $1::UUID, | ||
compartments = ARRAY[$1]::UUID[], | ||
content = jsonb_set(content::jsonb, '{meta, project}'::text[], to_jsonb($1), true) |
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.
Clever
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.
😅
compartments = ARRAY[$1]::UUID[], | ||
content = jsonb_set(content::jsonb, '{meta, project}'::text[], to_jsonb($1), true) | ||
WHERE | ||
"projectId" IS NULL OR "projectId" = $1`; |
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.
Might need to add AND "content"<>''
Kudos, SonarCloud Quality Gate passed! |
Use data migrations in version numbers (#2816) Fixes #2784 - Data migrations framework (#2786) Fixes #2817 - add jose to bot lambda layer (#2818) Move "public" resources into R4 Project (#2591) fix(auth/external): add some missing url params for external auth `PKCE` exchanges (#2812) Experiment: remove prettier from ESLint config, try out Biome (Rome fork) (#2748) Fixes #2808 - handle observation groups in DiagnosticReportDisplay (#2809) Adding content for Titan case study (#2806) QuestionnaireForm Cleanup (#2801) UI Clone for Lab Order Questionnaire (#2787) Lab order bot (#2791) Blog Post: GraphQL vs. Rest (#2781) Fixes #2761 - docs for bot logs and aws athena (#2779) Adding EMPI documentation (#2754) Example on Lab Ordering demo (#2716) Fixed links on about page (#2777) Adding blog material (#2776) Fixes #2771 - handle numeric return values from bots (#2775) New www 'about' page (#2770)
Use data migrations in version numbers (#2816) Fixes #2784 - Data migrations framework (#2786) Fixes #2817 - add jose to bot lambda layer (#2818) Move "public" resources into R4 Project (#2591) fix(auth/external): add some missing url params for external auth `PKCE` exchanges (#2812) Experiment: remove prettier from ESLint config, try out Biome (Rome fork) (#2748) Fixes #2808 - handle observation groups in DiagnosticReportDisplay (#2809) Adding content for Titan case study (#2806) QuestionnaireForm Cleanup (#2801) UI Clone for Lab Order Questionnaire (#2787) Lab order bot (#2791) Blog Post: GraphQL vs. Rest (#2781) Fixes #2761 - docs for bot logs and aws athena (#2779) Adding EMPI documentation (#2754) Example on Lab Ordering demo (#2716) Fixed links on about page (#2777) Adding blog material (#2776) Fixes #2771 - handle numeric return values from bots (#2775) New www 'about' page (#2770)
Creates a special Project for holding the base R4 FHIR specification conformance resources. The goal is to allow inclusion of those resources into other projects effectively via reference, so the hundreds of
StructureDefinition
and other resources do not live in each Project separately. This change includes:Repository
to add the R4Project
compartment to all DB read queries; blocking all writes across project boundariesRelated to #2581