Skip to content

Add repository and deployment links - #84

Merged
ShoGinn merged 2 commits into
mainfrom
codex/add-app-links
Aug 1, 2026
Merged

Add repository and deployment links#84
ShoGinn merged 2 commits into
mainfrom
codex/add-app-links

Conversation

@ShoGinn

@ShoGinn ShoGinn commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a GitHub repository link to the app navigation
  • add the live Railway deployment URL to the README
  • cover the new navigation link in the app shell test

Verification

  • frontend tests
  • frontend typecheck
  • frontend lint and formatting
  • pre-commit and pre-push hooks

Copilot AI review requested due to automatic review settings August 1, 2026 23:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds discovery links for the project by surfacing the repository in the app’s sidebar navigation and publishing the live Railway deployment URL in the README, with a small AppShell test update to cover the new navigation link.

Changes:

  • Add a prominent Railway deployment link near the top of README.md.
  • Add a GitHub repository link to the AppShell sidebar navigation.
  • Extend the AppShell test to assert the GitHub link and its href.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
README.md Adds a top-level link to the live Railway deployment.
frontend/src/app/AppShell.tsx Adds a GitHub repository link to the sidebar navigation.
frontend/src/app/AppShell.test.tsx Adds a test assertion for the new GitHub navigation link.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread frontend/src/app/AppShell.tsx Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Suppressed comments (3)

frontend/src/app/AppShell.tsx:103

  • This link opens a new tab via target="_blank" but the accessible name doesn’t indicate that behavior. Consider adding an aria-label that includes “opens in a new tab” so screen-reader users aren’t surprised.
              <a className="nav-link" href={githubUrl} target="_blank" rel="noopener noreferrer">

README.md:11

  • The README now presents a one-click link to the public deployment before the existing warning about not entering real financial data into untrusted deployments. To reduce the chance users miss the warning, consider adding a short caution immediately adjacent to the deployment link.
**[Open Tax Tracker on Railway](https://shoginn-tax-tracker.up.railway.app)**

frontend/src/app/AppShell.test.tsx:23

  • If the GitHub link’s accessible name is updated to mention opening a new tab, this test should query by the new accessible name to match user-facing semantics.
    expect(screen.getByRole("link", { name: "GitHub" })).toHaveAttribute(
      "href",
      "https://github.com/ShoGinn/tax-tracker",
    );

@ShoGinn
ShoGinn merged commit 430d8f6 into main Aug 1, 2026
16 checks passed
@ShoGinn
ShoGinn deleted the codex/add-app-links branch August 1, 2026 23:17
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.

2 participants