Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Tutorial]: Create Onchain Subscription Payments with Spend Permissions #1410

Merged
merged 17 commits into from
Dec 20, 2024
Merged

[Tutorial]: Create Onchain Subscription Payments with Spend Permissions #1410

merged 17 commits into from
Dec 20, 2024

Conversation

hughescoin
Copy link
Collaborator

What changed? Why?
New tutorial for spend permissions onchain subscription use case

Notes to reviewers

How has it been tested?

@vercel Vercel
Copy link

vercel bot commented Dec 17, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
base-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 7:37pm
web-base-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 7:37pm

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Dec 17, 2024 •
edited
Loading

✅ 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
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

@wbnns wbnns added the base docs review To be reviewed by the Base docs team label Dec 18, 2024
ericbrown99
ericbrown99 reviewed Dec 18, 2024
View reviewed changes
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
apps/base-docs/tutorials/docs/1_smart-wallet-spend-permissions.md Outdated Show resolved Hide resolved
@hughescoin hughescoin enabled auto-merge (squash) December 20, 2024 19:29
@cb-heimdall
Copy link
Collaborator

Review Error for ericbrown99 @ 2024-12-20 19:45:50 UTC
User failed mfa authentication, see go/mfa-help

@hughescoin hughescoin merged commit 76de6b6 into master Dec 20, 2024
9 checks passed
@hughescoin hughescoin deleted the spendpermissions-tutorial branch December 20, 2024 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericbrown99 ericbrown99 ericbrown99 approved these changes

@wbnns wbnns wbnns approved these changes

Assignees
No one assigned
Labels
base docs review To be reviewed by the Base docs team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hughescoin @cb-heimdall @wbnns @ericbrown99