Add hint for export workflow

[ronjakrg]

Description

Changed the url of GitHub logo
Added message after a workflow has been exported
Added button to go back from managing databases

Changes

navbar.html has new url
views.py now imports new modules and has a message which indicates a successful export and shows the destination path
databases.html has new button

Testing

PR checklist

Development

• If necessary, I have updated the documentation (README, docstrings, etc.)
• If necessary, I have created / updated tests.

Mergeability

• main-branch has been merged into local branch to resolve conflicts
• The tests and linter have passed AFTER local merge
• The code has been formatted with black

Code review

• I have self-reviewed my code.
• At least one other developer reviewed and approved the changes

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
protzilla
run.py
run_helper.py
runner.py
protzilla/data_analysis
differential_expression.py
differential_expression_helper.py					125, 143
differential_expression_kruskal_wallis.py					227
differential_expression_linear_model.py
differential_expression_mann_whitney.py					266
differential_expression_t_test.py
protein_graphs.py
ptm_analysis.py					56, 123-125
protzilla/data_integration
di_plots.py
protzilla/data_preprocessing
outlier_detection.py
transformation.py
protzilla/importing
ms_data_import.py					122, 276
protzilla/methods
data_analysis.py					21, 174, 250, 283, 308, 351, 916, 935
data_preprocessing.py
importing.py					144
protzilla/utilities
transform_dfs.py
utilities.py
ui/main
settings.py
ui/runs
fields.py
views.py					1, 21, 391
views_helper.py
ui/runs/forms
data_analysis.py					318-322, 379, 409, 445, 468, 1183, 1221, 1243
Project Total

The report is truncated to 25 files out of 54. To see the full report, please visit the workflow summary page.

This report was generated by python-coverage-comment-action

[gitjannes]

statt onclick sollte auch nur href="/" genügen, dass wäre dann auch mit line 4 in navbar.html konsistent sein.

Suggested change

	<button id="back-button" type="button" class="btn btn-red mr-auto" onclick="window.location.href='http://127.0.0.1:8000/';">
	<button id="back-button" type="button" class="btn btn-red mr-auto" href="/">

[ronjakrg]

Habe gerade mal nachgeschaut das href="/" scheint nur mit dem <a>-Tag zu gehen und wurde so auch z. B. in details.html verwendet. Passe das entsprechend an, danke!

[gitjannes]

Sieht gut aus 👍
Wegen dem requested change in databases.html, ich persönlich finde die kurze Variante cleaner, ich kann mir auch vorstellen, dass die statische lokale Adresse Probleme bereiten könnte, sollte Protzilla nicht dort laufen.

[gitjannes]

Top, passt meiner Meinung nach 👍. Wäre natürlich cool, wenn dann ein ähnlicher Footer wie der von Lennard für die databases.html verwendet wird, aber da muss er ja auch erstmal fertig sein.

[noLeonardo]

Schöner Code und funktioniert auch alles :)
Beim Back-Button könnten wir die Positionierung vielleicht nochmal diskutieren, wenn wir ein einheitliches Design-Konzept haben. Ich persönlich finde es direkt am linken Rand und/ oder unten ein bisschen intuitiver, aber von der Funktionsweise top.