Uh oh!
There was an error while loading. Please reload this page.
[Fix] Support links open the wrong GitHub repository - #77
Conversation
No actionable issues found. See task |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
edelauna
left a comment
There was a problem hiding this comment.
Approved with some minor suggestions
| @@ -0,0 +1,8 @@ | |||
| export const EXTERNAL_LINKS = { | |||
| GITHUB_REPO: "https://github.com/Zoo-Code-Org/Zoo-Code", | |||
There was a problem hiding this comment.
Would it be worth deriving the issue/template/security URLs from a single repo base URL here? That would keep future repo moves from needing several string updates in the same constants file.
| it("links support users to the Zoo Code GitHub repository", () => { | ||
| render(<Announcement hideAnnouncement={vi.fn()} />) | ||
| expect(screen.getByRole("link", { name: "GitHub" })).toHaveAttribute("href", EXTERNAL_LINKS.GITHUB_REPO) |
There was a problem hiding this comment.
Since this assertion imports the same constant as the component, would a small constants test with the literal Zoo Code URLs give better coverage for accidental target changes?
9faf3ae to
4131a7bCompareedelauna
commented
May 13, 2026
Going to merge this so that we can focus on remaining changes in #85 |
Uh oh!
There was an error while loading. Please reload this page.
What problem this solves
Fixes an issue where users in the VS Code webview could still be sent to the old
RooCodeInc/Roo-Coderepository for bug reports, feature requests, marketplace issue reporting, and the security policy after the Zoo Code handoff.Why this change was made
The webview still had several hardcoded GitHub support links pointing at the upstream repository. This centralizes those URLs in one shared constants module and updates the affected support surfaces to use
Zoo-Code-Org/Zoo-Code.User impact
Support links in the About screen, Marketplace footer, announcement modal, and error fallback now open the Zoo Code repository and issue templates instead of the old upstream repo.