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

Show Annif version in WebUI #745

Merged
merged 2 commits into from
Nov 21, 2023
Merged

Show Annif version in WebUI #745

merged 2 commits into from
Nov 21, 2023

Conversation

juhoinkinen
Copy link
Member

@juhoinkinen juhoinkinen commented Nov 20, 2023

It would be nice to know which version of Annif is running for the Web UI.

This PR adds a version number string, e.g. Annif v1.1.0 (or Annif v1.1.0.dev0 for the development version) outside the text box in the lower right corner, see screenshot below.

An alternative position for the version number could be just below "Web UI" title in the header, but the position of this PR is the same as will be in Finto AI.

image

@juhoinkinen juhoinkinen added this to the 1.1 milestone Nov 20, 2023
Copy link

codecov bot commented Nov 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (86ce3e7) 99.67% compared to head (e860927) 99.67%.

❗ Current head e860927 differs from pull request most recent head 32be883. Consider uploading reports for the commit 32be883 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #745   +/-   ##
=======================================
  Coverage   99.67%   99.67%           
=======================================
  Files          89       89           
  Lines        6404     6404           
=======================================
  Hits         6383     6383           
  Misses         21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@osma
Copy link
Member

osma commented Nov 20, 2023

Looks very good, but is it clear enough with only the version number? Would it be better to say "Annif v1.1.0" or something along those lines...

Copy link

sonarcloud bot commented Nov 20, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@juhoinkinen juhoinkinen merged commit a501e53 into main Nov 21, 2023
16 checks passed
@juhoinkinen juhoinkinen deleted the show-annif-version-in-webui branch November 21, 2023 08:03
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.

2 participants