Fom MANE overview, link out to gene overview #368
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #368 +/- ##
==========================================
+ Coverage 90.18% 93.13% +2.95%
==========================================
Files 30 31 +1
Lines 1457 1501 +44
==========================================
+ Hits 1314 1398 +84
+ Misses 143 103 -40
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
| @@ -27,6 +27,8 @@ <h2 class="accordion-header" id="flush-headingOne"> | |||
| <!-- hidden fields passed from previous query --> | |||
| <form name="customizeForm" action="{{url_for('mane_overview')}}" method="post"> | |||
| <input type="hidden" name="build" value="GRCh38"/> | |||
| <input type="hidden" name="default_level" id="defaultLevel" value="{{extras.default_level}}"/> | |||
There was a problem hiding this comment.
Default level is not used in the MANE overview report, but in the gene overview, and this is a way to pass along the value
|
Looks like it is working!
On the gene page. Would it be possible to show what transcript is MANE as well? As the ID type differs, the users will not immediately see which one was "NM_001101.5" in the example above. |
Yes, I'll add it, but on my new PR, the one adding also exons. It will be easier |
|
Merging this one. Thanks @Jakob37! |
Description
Added/Changed/Fixed
How to test
Review
This version is a