Skip to content

feat: rc5.0.3 - #259

Open
karlnyr wants to merge 3 commits into
masterfrom
rc5.0.3
Open

feat: rc5.0.3#259
karlnyr wants to merge 3 commits into
masterfrom
rc5.0.3

Conversation

@karlnyr

@karlnyr karlnyr commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request updates the alignment_page.html to align with SWEDAC requirements.

Layout and formatting improvements:

  • Refactored the HTML structure for better indentation and readability, especially within the summary and sample overview panels. [1] [2]

Conditional rendering:

  • Added a conditional display of the SWEDAC logo on the summary page if the topsample.application_tag contains 'MW'.

Table improvements:

  • Updated the "Antal Reads" column header to "Antal Reads (**)" for clarity.
  • Improved handling of missing total_reads values by ensuring a value of 0 is displayed in a consistent format.

Minor formatting fixes:

  • Added missing <br> tags and improved whitespace for better visual separation in contact and customer information sections.
  • Removed unnecessary blank lines and improved the structure of table header rows.

Primary function of PR

  • Hot-fix
  • Patch
  • Minor functionality improvement
  • New type of analysis
  • Backward-breaking functionality improvement
  • This change requires internal documents to be updated
  • This change requires another repository to be updated

Testing

  • bash /home/proj/production/servers/resources/hasta.scilifelab.se/install-microsalt-stage.sh BRANCHNAME
  • us
  • conda activate S_microSALT
  • microSALT analyse --input /path/to/fastq/ SAMPLEINFO_FILE

Test results

These are the results of the tests, and necessary conclusions, that prove the stability of the PR.

Sign-offs

  • Approved to run at Clinical-Genomics by @karlnyr or @Clinical-Genomics/micro

@karlnyr
karlnyr requested a review from a team as a code owner September 3, 2026 08:43
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