-
Notifications
You must be signed in to change notification settings - Fork 479
add builder code faq #1022
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add builder code faq #1022
Conversation
✅ Heimdall Review Status
|
| ### What are the benefits of supporting Builder Codes? | ||
|
|
||
| - **Rewards:** If your app drives transactions, Builder Codes let Base automatically attribute that usage back to you, unlocking rewards as our program expands. | ||
| - **Analytics:** Developers can reliably track onchain usage, user acquisition, and conversion metrics in Base.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - **Analytics:** Developers can reliably track onchain usage, user acquisition, and conversion metrics in Base.dev | |
| - **Analytics:** Developers can reliably track onchain usage, user acquisition, and conversion metrics in Base.dev. |
|
|
||
| - Find your transaction hash | ||
| - View the input data field | ||
| - Verify the last 16 bytes are the ERC marker (`0x8021...8021`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Prefixing this with 0x might add a bit of confusion since the actual calldata will have other data before that. I would replace this with ...80218021 or something like that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I think I'd just say something like "Verify that the last 16 bytes of data are 8021 repeating."
Approved review 3730977415 from roethke is now dismissed due to new commit. Re-request for approval.
What changed? Why?
Notes to reviewers
How has it been tested?