Skip to content

Conversation

Bvvvp009
Copy link
Contributor

What changed? Why?
Changed in https://docs.base.org/learn/onchain-app-development/cross-chain/bridge-tokens-with-layerzero
Fixed a broken links and Removed old unsupported Goerli changed to Base sepolia example in the Learn →Onchain-app-development →Cross-chain → Bridge tokens with layerzero section. The original URL pointed to a non-existent page; it now correctly links to: Layerzero relevant sesctions.
Notes to reviewers
This fix sure changed broken links and as most of the OApp example follwed the latest layerzero V2 documentation i have changed chain to Sepolia.
How has it been tested?
Tested all links one by one to make sure they are correct and pointing to relevent sections.

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Aug 25, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@Bvvvp009
Copy link
Contributor Author

@soheimam @ericbrown99

@youssefea
Copy link
Contributor

Thank you ! but cant merge as your commits must be verified

@youssefea youssefea closed this Oct 2, 2025
@youssefea
Copy link
Contributor

Please verify commits and submit another PR

Bvvvp009 added a commit to Bvvvp009/base-docs that referenced this pull request Oct 5, 2025
- Update LayerZero documentation links to V2 structure
- Replace Base Goerli/Optimism Goerli with Base Sepolia/Optimism Sepolia
- Update RPC URLs, endpoint addresses, and endpoint IDs for Sepolia testnets
- Fix broken LayerZero endpoint documentation links
- Update Foundry documentation link to new structure

Addresses the exact changes from PR base#273 with verified commits.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants