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

Move the Monarch KG Release process page to the Monarch-app documentation #820

Open
2 of 3 tasks
sagehrke opened this issue Sep 24, 2024 · 3 comments · Fixed by #893
Open
2 of 3 tasks

Move the Monarch KG Release process page to the Monarch-app documentation #820

sagehrke opened this issue Sep 24, 2024 · 3 comments · Fixed by #893
Assignees
Labels
good first issue Good for newcomers

Comments

@sagehrke
Copy link
Member

sagehrke commented Sep 24, 2024

Deploy this:

Thank you!

@sagehrke sagehrke added the good first issue Good for newcomers label Sep 24, 2024
@sagehrke sagehrke changed the title Publish the Monarch-app documentation page Move the Monarch KG Release process page to the Monarch-app documentation Nov 1, 2024
@ptgolden ptgolden mentioned this issue Nov 5, 2024
1 task
kevinschaper pushed a commit that referenced this issue Nov 7, 2024
### Related issues

- Removes block from #820 


### Summary

- Updates `typer` to 0.12.0
- Moves CLI to new `Annotated` API
- Removes duplicated function signatures in `cli.py` that are also in
`solr_cli.py`

### Checks

- [x] All tests have passed (or issues created for failing tests)

---------

Co-authored-by: Patrick Golden <[email protected]>
@ptgolden ptgolden mentioned this issue Nov 15, 2024
1 task
@ptgolden
Copy link
Member

ptgolden commented Nov 15, 2024

Make sure this page is linked as the main documentation site for the monarch-app repo both in the "About" section and "readme" section of the repo.

@sagehrke Can you specify what you mean by "this page" and "the repo"?

@sagehrke
Copy link
Member Author

Sure! In this repo (the Monarch-app repo) we want to make sure the updated documentation site is clearly findable. I am hoping you can confirm it is listed in the About section of the repo (if possible) as well as the Readme.

@ptgolden
Copy link
Member

ptgolden commented Nov 15, 2024

After we get an OK from @kevinschaper on #893, the last thing to do will be to address the last task on that list. @sagehrke and I discussed it this afternoon, and she agreed to edit the readme to

  1. Link to https://monarch-app.monarchinitiative.org/ from the documentation badge, rather than the consortium documentation
  2. Change the copy on the introductory paragraph to more accurately reflect what is in this repository

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants