Skip to content

fix: update FaceID wallet redirects and strip anchors in redirect validation#22505

Merged
critesjosh merged 1 commit intonextfrom
critesjosh/fix-redirect-targets
Apr 16, 2026
Merged

fix: update FaceID wallet redirects and strip anchors in redirect validation#22505
critesjosh merged 1 commit intonextfrom
critesjosh/fix-redirect-targets

Conversation

@critesjosh
Copy link
Copy Markdown
Contributor

Summary

  • Update three stale FaceID wallet redirect targets to point to /developers/docs/tutorials/contract_tutorials/counter_contract instead of the generic tutorials index
  • Fix redirect validation script to strip #anchor fragments before path validation so targets with anchors don't falsely fail

Test plan

  • Run docs/scripts/validate_redirect_targets.sh and confirm no false positives from anchored paths
  • Verify the three updated redirects resolve correctly on a deploy preview

…idation

Update stale FaceID wallet redirect targets to point to the counter
contract tutorial instead of the generic tutorials page. Fix redirect
validation script to strip fragment identifiers before checking paths.
@critesjosh critesjosh marked this pull request as ready for review April 13, 2026 13:51
@critesjosh critesjosh enabled auto-merge April 16, 2026 18:36
@critesjosh critesjosh added this pull request to the merge queue Apr 16, 2026
Merged via the queue into next with commit 6e906f0 Apr 16, 2026
39 checks passed
@critesjosh critesjosh deleted the critesjosh/fix-redirect-targets branch April 16, 2026 19:12
AztecBot pushed a commit that referenced this pull request Apr 16, 2026
…idation (#22505)

## Summary
- Update three stale FaceID wallet redirect targets to point to
`/developers/docs/tutorials/contract_tutorials/counter_contract` instead
of the generic tutorials index
- Fix redirect validation script to strip `#anchor` fragments before
path validation so targets with anchors don't falsely fail

## Test plan
- [ ] Run `docs/scripts/validate_redirect_targets.sh` and confirm no
false positives from anchored paths
- [ ] Verify the three updated redirects resolve correctly on a deploy
preview
@AztecBot
Copy link
Copy Markdown
Collaborator

✅ Successfully backported to backport-to-v4-next-staging #22580.

Thunkar added a commit that referenced this pull request Apr 17, 2026
BEGIN_COMMIT_OVERRIDE
fix(pxe): cap event filter toBlock to last synced block (#22573)
fix(pxe): round tx expiration timestamp to reduce precision (#22577)
fix: eliminate anvil watcher warp race and false success logs (#22584)
refactor: aztec new and init creating 2 crates (#20681)
test: aztec new scaffold works (#20711)
feat(cli): warning if contract crate has tests (#20723)
feat(cli): auto-recompiling when aztec test is run (#20729)
feat: aztec new supporting multiple contract crates (#21007)
feat: asserts that aztec dep version matches cli (#21245)
chore: backport aztec CLI improvements to v4-next (#22587)
feat: check noir release has nargo binaries before releasing (#22551)
chore: cache chainInfo in embeddedwallet (#22592)
fix: wrap external getCapsule in transactionAsync (#22595)
fix(pxe): throw clear error for invalid comparator in pick_notes
(#22585)
refactor(aztec-nr): rename conversion fns to encode_/decode_ naming
(#22576)
feat: infrastructure for testing `[new_contract_artfiacts,
old_aztec_stack]` (#22593)
chore: fix unnecessary and inconsistent side-effect counter increments
(#22245)
fix: update FaceID wallet redirects and strip anchors in redirect
validation (#22505)
docs: add getting started on testnet guide (#22366)
docs: add getting started on testnet guide (backport #22366) (#22619)
feat(aztec-nr): new BoundedVec emit private log APIs (#22064)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants