Skip to content

Show the app version in the site footer - #1868

Merged
KOliver94 merged 2 commits into
mainfrom
version-tag
Oct 20, 2026
Merged

KOliver94 merged 2 commits into
mainfrom
version-tag

Conversation

@KOliver94

Copy link
Copy Markdown
Member

Summary

The site footer now shows which version is running, next to the copyright line, like Backstage does. Tagged builds show v1.5.0 and link to that GitHub release; builds from main show the short commit hash and link to the commit. Local dev and tests show nothing.

The link opens in a new tab with the same arrow as the footer's other external links, and screen readers read it as "Verzió: v1.5.0".

How the version gets there

No workflow or Dockerfile change. The image build already passes SENTRY_RELEASE (request-manager@<version> on tags, the commit SHA otherwise) for the Sentry plugin. Vite now also exposes it to the app as import.meta.env.RELEASE, and the footer turns it back into a release or commit link.

Notes for reviewers

  • The link depends on the release naming in docker-publish.yml: the request-manager@ prefix, and the v that the workflow strips and the footer adds back. If that naming changes, the link breaks without any build error.
  • Only the public site shows the version; the admin has no footer.
  • I checked it in a production build served locally, built once with a tag release name and once with a commit SHA, at desktop and phone width.

@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 16 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: cf704472-dfae-4063-bd3f-1f74f63067fc
📥 Commits

Reviewing files that changed from the base of the PR and between d1f61e2 and 6fa60a7.

📒 Files selected for processing: 3
  • frontend/src/site/components/SiteFooter.tsx
  • frontend/src/vite-env.d.ts
  • frontend/vite.config.mts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.35%. Comparing base (d1f61e2) to head (6fa60a7).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
frontend/src/site/components/SiteFooter.tsx 0.00% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1868      +/-   ##
==========================================
- Coverage   43.41%   43.35%   -0.06%     
==========================================
  Files         282      282              
  Lines        7267     7277      +10     
  Branches     1459     1464       +5     
==========================================
  Hits         3155     3155              
- Misses       4091     4101      +10     
  Partials       21       21              
Flag Coverage Δ
backend 97.73% <ø> (ø)
frontend 5.08% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@KOliver94
KOliver94 enabled auto-merge (squash) October 20, 2026 11:44
@KOliver94
KOliver94 merged commit a402223 into main Oct 20, 2026
15 checks passed
@KOliver94
KOliver94 deleted the version-tag branch October 20, 2026 11:46
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.

1 participant