Conversation
… /address/:address Existing links of the form /#/lookup/0x... are preserved via a small hash-rewrite shim in index.html that runs before React boots, plus a client-side redirect from /lookup/:address to /address/:address. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…ress Header "Lookup" link now points at /address. /lookup and /lookup/:address (non-hash) and #/lookup, #/lookup/<addr> (hash) all redirect to /address[/<addr>]. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Migrate from v1 checkAllByAddresses to v2 /v2/contract/all-chains/{address};
404 treated as empty result rather than error
- Add PageLayout stacked-card shell and MatchBadge pill components
- Rewrite Lookup/Field: validate on submit only, inline error, no double-fetch
- Rewrite Lookup/Result: bordered table with Creation/Runtime Match columns,
env-driven Sourcify Repo link, responsive mobile stacked-card rows
- Fix back-to-search race condition using useRef guard instead of useState
- Fix tooltip z-index hidden behind stacked card
- Fix horizontal table scroll (was blocked by overflow-hidden)
- Fix react-device-detect dependency; use Tailwind responsive classes instead
- Add ceruleanBlue.50 token for page background tint
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Auto-search on valid address input; remove Look up button - Show inline error on blur for invalid address input - Reset page state when navigating to /address with no param - Contain table horizontal scroll within card (overflow-x-hidden on page wrapper) - Make mobile table horizontally scrollable instead of stacked cards - Remove bottom Lookup/Verify CTA buttons - Link Verify Contract to REACT_APP_VERIFY_URL instead of internal /verifier route - Remove "Do you have the source code and metadata?" → "source code" - Add VERIFY_URL to constants.ts - Tooltip max-width 16rem and smaller font Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
- Show invalid address error at 42 chars without needing to blur - Equal-width stacked buttons in Source column - Remove Match ID column Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Replace HashRouter with BrowserRouter; redesign /address page with Sourcify v2 API
✅ Deploy Preview for sourcify-ui ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
manuelwedler
reviewed
May 6, 2026
Member
manuelwedler
left a comment
manuelwedler
left a comment
There was a problem hiding this comment.
https://staging.sourcify.dev/#/lookup/0x1F98431c8aD98523631AE4a59f267346ea31F984
When I try to open the redirect, it forwards me to the /address URL but the page stays white. Tried different browser contexts. Is it just for me or do you have it too?
Member
Author
|
@manuelwedler good catch, same for me too. This should've been tested as part of that PR. Let me address this first |
Set homepage to "/" so Create React App emits /static/... instead of ./static/... for asset URLs. Required for the BrowserRouter migration: the legacy hash shim in index.html does history.replaceState() before React boots, which changes the document URL that subsequent relative asset paths resolve against — breaking JS/CSS loading on /#/lookup/<address>. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Use absolute asset paths to fix /#/lookup legacy redirect
manuelwedler
approved these changes
May 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.