Skip to content

Rems spl updated#275

Open
violetcrestedwren wants to merge 14 commits into
masterfrom
rems_spl_updated
Open

Rems spl updated#275
violetcrestedwren wants to merge 14 commits into
masterfrom
rems_spl_updated

Conversation

@violetcrestedwren

Copy link
Copy Markdown
Collaborator

No description provided.

@dkrylovsb dkrylovsb self-assigned this May 1, 2025
@dkrylovsb dkrylovsb self-requested a review May 1, 2025 18:36
Comment thread src/constants/fields/drugremsspl.yaml
Comment thread src/pages/apis/drug/remsspl/_explorers.yaml
Comment thread src/pages/apis/drug/remsspl/_explorers.yaml Outdated
Comment thread src/pages/apis/drug/remsspl/_infographics.yaml
Comment thread src/pages/apis/drug/remsspl/_infographics.yaml Outdated
Comment thread src/components/RenderContentObject/KeyFacts.jsx Outdated

@dkrylovsb dkrylovsb left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added comments inline

@dkrylovsb

Copy link
Copy Markdown
Collaborator

@violetcrestedwren Also need to merge latest master in

@dkrylovsb

Copy link
Copy Markdown
Collaborator

@violetcrestedwren This sample query brings back records without medication guide:
image

@dkrylovsb

Copy link
Copy Markdown
Collaborator

@violetcrestedwren This query returns records for unrelated sponsors:
image

@dkrylovsb

Copy link
Copy Markdown
Collaborator

@violetcrestedwren Not all REMS records have openfda. Here we should use products.manufactured_product.brand_name, which is populated for every REMS record:
image

# Conflicts:
#	package.json
#	src/components/ApiStatus.tsx
#	src/components/ApiUsage.jsx
#	src/components/DataDictionary.jsx
#	src/components/EndpointBox.tsx
#	src/components/FieldDownload.tsx
#	src/components/FieldsHarmonization.tsx
#	src/components/RenderContentObject/KeyFacts.jsx
#	src/constants/fields/master_fields.yaml
#	src/pages/apis/doc-links.yaml
#	src/pages/apis/drug/index.tsx
#	src/pages/data/downloads/index.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants