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 AMR comparison index view and link it in the menu and home page. #35

Merged
merged 16 commits into from
Jan 15, 2024

Conversation

njohner
Copy link
Contributor

@njohner njohner commented Jan 10, 2024

With this PR I add a comparison index view for the AMR analyses and add a link to it in the menu and on the home page. I also merge the three tabular comparisons (gene, class and subclass presence/absence) into one view with a selection field to choose which type of comparison to make. See screenshots below.

amr_home_and_menu
amr_index_comp

amr_comparison

@njohner njohner changed the title Nj/amr home Add ARM analyses to menu and to home page. Jan 10, 2024
@njohner njohner changed the title Add ARM analyses to menu and to home page. Add AMR comparison index view and add a link to it in the menu and home page. Jan 10, 2024
@njohner njohner changed the title Add AMR comparison index view and add a link to it in the menu and home page. Add AMR comparison index view and link it in the menu and home page. Jan 10, 2024
@njohner njohner force-pushed the nj/amr_home branch 2 times, most recently from e627626 to da45c9d Compare January 10, 2024 10:41
@njohner njohner marked this pull request as ready for review January 10, 2024 10:41
@njohner njohner requested a review from tpillone January 10, 2024 10:41
Capitalization in URLs is unusual and makes things more complex.
Moreover it was not used consistently, so I decided to drop it.
I also fix the indentation to improve readability.
We add a field to the form to choose the type of comparison to
make.
This makes using the same templates for the comparison index view
easier.
@njohner njohner merged commit 092a27f into master Jan 15, 2024
@njohner njohner deleted the nj/amr_home branch January 15, 2024 07:17
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.

1 participant