Skip to content

Update footer to be bootstrap 5 compatible for multiqc 1.32#24

Merged
aanil merged 6 commits into
NationalGenomicsInfrastructure:masterfrom
aanil:master
Jan 7, 2026
Merged

Update footer to be bootstrap 5 compatible for multiqc 1.32#24
aanil merged 6 commits into
NationalGenomicsInfrastructure:masterfrom
aanil:master

Conversation

@aanil
Copy link
Copy Markdown
Member

@aanil aanil commented Dec 17, 2025

No description provided.

Comment thread multiqc_ngi/templates/ngi/footer.html
@aanil aanil requested a review from ssjunnebo January 7, 2026 13:20
Comment thread multiqc_ngi/templates/ngi/footer.html Outdated
<a href="http://search.swedac.se/en/accreditations/1239" target="_blank" class="pull-right">
<img src="data:image/png;base64,{{ include_file('assets/img/Swedac-logo-uppsala.png', b64=True) }}">
</a>
{% set accreditation = {"stockholm": {"id": "1850", "logo": "Swedac-logo-stockholm.png"}, "uppsala": {"id": "1239", "logo": "Swedac-logo-uppsala.png"}} %}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have an internal audit deviation about this (#69), which suggests to not have the swedac logo being implemented into the MultiQC report:
"At the end of MultiQC reports there is a Swedac logo and text referring to the accreditation.

Suggestion: Remove the logo and text at the end of the MultiQC report and make sure this does not occur elsewhere as this may be misleading and imply accreditation where it does not apply. We only want the final report to include the logo. "

Has this been addressed here? To me it doesn't look like it.

@matrulda Is this something that you guys in Uppsala need to consider as well?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the Swedac logo and text.

@aanil aanil merged commit 8038674 into NationalGenomicsInfrastructure:master Jan 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants