Skip to content

Add deployed commit link and voice-wave favicon to the calculator - #46

Merged
szhaomsft merged 2 commits into
mainfrom
feat/cost-calculator-commit-link
Oct 22, 2026
Merged

szhaomsft merged 2 commits into
mainfrom
feat/cost-calculator-commit-link

Conversation

@szhaomsft

Copy link
Copy Markdown
Collaborator

Summary

  • Add a footer link displaying the short source commit hash for each GitHub Pages deployment, with a full-hash tooltip and a link to the exact commit.
  • Stamp the deployed SHA from GitHub Actions metadata; show an explicit local-preview label when not deployed and fail deployment if the expected footer marker is missing.
  • Add an original green voice-wave SVG favicon matching the calculator theme, wire it into the HTML, and serve it locally with the correct SVG MIME type.
  • Publish only the six required browser assets, including the favicon; keep server code and other repository files out of the Pages artifact.
  • Correct the remaining stale methodology text to match the previously updated 10 input / 20 output audio tokens-per-second defaults.

Validation

  • All 22 calculator tests pass; server syntax and whitespace checks pass.
  • Executed the workflow's actual staging script and verified the short hash, full commit URL, SVG, favicon reference, and exact six-file artifact allowlist.
  • Confirmed favicon GET and HEAD requests return HTTP 200 with image/svg+xml content type.

Deployment

The existing Pages workflow publishes after merge into main at https://microsoft.github.io/voice-agent/ .

Based on current main; unrelated local files are excluded.

szhaomsft and others added 2 commits October 22, 2026 15:36
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@szhaomsft
szhaomsft merged commit e247824 into main Oct 22, 2026
3 checks passed
@szhaomsft
szhaomsft deleted the feat/cost-calculator-commit-link branch October 22, 2026 07:39
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