Skip to content

fix: correct bounty submission navigation#907

Open
jakearmstrong59 wants to merge 3 commits intoentrius:testfrom
jakearmstrong59:fix/issue-submission-author-links
Open

fix: correct bounty submission navigation#907
jakearmstrong59 wants to merge 3 commits intoentrius:testfrom
jakearmstrong59:fix/issue-submission-author-links

Conversation

@jakearmstrong59
Copy link
Copy Markdown

@jakearmstrong59 jakearmstrong59 commented Apr 30, 2026

Summary

Fixes #906.

Corrects navigation from bounty issue submission tables and keeps the return button simple and predictable.

Changes

  • Restore SPA navigation for author links in the issue details Submissions table.
  • Prevent LinkBox and LinkTableRow callers from overriding their internal anchor/link behavior accidentally.
  • Preserve the bounty details return path when opening a submitted PR or miner from the bounty submissions table.
  • Use a simple Back label for bounty submission, Discoveries, and Leaderboard navigation flows.

Testing

  • npm run build
  • Focused ESLint checks
  • Manually verified bounty author link navigation
  • Manually verified submitted PR navigation and Back
  • Manually verified Discoveries miner navigation and Back

Fix

chrome_tzLiyJas6c.mp4

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label Apr 30, 2026
@jakearmstrong59 jakearmstrong59 changed the title fix: restore issue submission author links fix: correct bounty submission navigation Apr 30, 2026
@jakearmstrong59 jakearmstrong59 force-pushed the fix/issue-submission-author-links branch from d58a7ba to 857d709 Compare April 30, 2026 20:29
@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 9, 2026

Fix conflicts.

…into fix/issue-submission-author-links

# Conflicts:
#	src/components/issues/IssueSubmissionsTable.tsx
#	src/pages/TopMinersPage.tsx
@jakearmstrong59
Copy link
Copy Markdown
Author

Hi @anderdc I've fixed conflicts. Thanks for reviewing.

@ventura-oss
Copy link
Copy Markdown
Contributor

Could you please resolve the merge conflicts so we can proceed with the review?

@jakearmstrong59
Copy link
Copy Markdown
Author

@anderdc Please review again. I solved conflicts. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bounty submission links and back navigation are inconsistent

3 participants