Skip to content
This repository was archived by the owner on Feb 10, 2025. It is now read-only.

update checker app to support internal navigation #75

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

veganbeef
Copy link

@veganbeef veganbeef commented Dec 6, 2024

This PR updates:

  • CheckerProvider.tsx to handle internal navigation with a bypass for external navigation
  • checkerReducer.ts to set internal navigation history
  • type files to accomodate navigation changes

The end result is that the browser "back" button will work as expected within the Checker app.

closes PAR-689

Copy link

vercel bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitcoin-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 6, 2025 2:01pm

hussedev
hussedev previously approved these changes Jan 3, 2025
Copy link
Member

@hussedev hussedev left a comment

Choose a reason for hiding this comment

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

LGTM! Nice Job

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants