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

Added images to README.md for #73 #76

Closed
wants to merge 9 commits into from
Closed

Added images to README.md for #73 #76

wants to merge 9 commits into from

Conversation

turbomam
Copy link
Collaborator

NMDC image may be too big

Still need to find newer/better? schema image or validate the current one.

@turbomam turbomam marked this pull request as ready for review November 30, 2021 16:53
@wdduncan
Copy link
Contributor

@turbomam This PR failed the validation check. Given how long it has been open is it best to just close the PR at this point?

@turbomam
Copy link
Collaborator Author

@turbomam This PR failed the validation check. Given how long it has been open is it best to just close the PR at this point?

Sure. I think @cmungall wanted be to add some images back into the README.md after some refactoring had cause broken image links.

The badges and slideshow link are good.

What do you think of adding an NMDC banner? I think I had on in this old PR, but it was awfully big.

@wdduncan
Copy link
Contributor

I think the NMDC image is fine, although it might need to be reduced.

@turbomam
Copy link
Collaborator Author

I decided to include the image but not the "high level overview" diagram

@turbomam turbomam closed this Aug 22, 2022
turbomam added a commit that referenced this pull request Feb 21, 2024
…vant_protocols

add migrator for moving relevant_protocols to protocol_link PR21
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.

None yet

2 participants