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

💄 Add the phenotype condition module diagram #105

Merged
merged 4 commits into from
Jan 27, 2025
Merged

Conversation

brendagutman
Copy link
Collaborator

Motivation
Phenotype Condition module has a diagram which needs to be updated to use in the module homepage.

Approach
Insert the image into storage and place into home page for module.

@github-actions github-actions bot temporarily deployed to pull request January 25, 2025 01:14 Inactive
Copy link

github-actions bot commented Jan 25, 2025

🚀 IG Site Preview Deployed

Latest commit: 9d12b5c

View deployment

@github-actions github-actions bot temporarily deployed to pull request January 27, 2025 16:18 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 27, 2025 16:33 Inactive
@brendagutman brendagutman requested a review from torstees January 27, 2025 16:47
Copy link
Collaborator

@torstees torstees left a comment

Choose a reason for hiding this comment

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

Can you move the ResourceType references under the profile names (NCPI Participant, NCPI Condition)?

Copy link
Collaborator

@torstees torstees left a comment

Choose a reason for hiding this comment

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

looks great!

@brendagutman brendagutman merged commit eb550b9 into main Jan 27, 2025
2 checks passed
@brendagutman brendagutman deleted the bg/fd_2009 branch January 27, 2025 23:53
@github-actions github-actions bot temporarily deployed to pull request January 27, 2025 23:54 Inactive
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