Skip to content

fix(spec)!: deprecate paymentPointer in favor of walletAddress#520

Open
sidvishnoi wants to merge 1 commit into
mainfrom
spec-walletAddress
Open

fix(spec)!: deprecate paymentPointer in favor of walletAddress#520
sidvishnoi wants to merge 1 commit into
mainfrom
spec-walletAddress

Conversation

@sidvishnoi

@sidvishnoi sidvishnoi commented Apr 10, 2025

Copy link
Copy Markdown
Collaborator

Closes #439


Preview | Diff
(pr-preview is not working #511 (comment), so manually created above)

@netlify

netlify Bot commented Apr 10, 2025

Copy link
Copy Markdown

Deploy Preview for webmonetization-preview ready!

Name Link
🔨 Latest commit ad906e1
🔍 Latest deploy log https://app.netlify.com/sites/webmonetization-preview/deploys/67f79f63ca391b00088a7f54
😎 Deploy Preview https://deploy-preview-520--webmonetization-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is re-generated with ./scripts/respec.sh.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I'll set up automation for this file first 😄
Doesn't make sense to add this in PRs.

@sidvishnoi
sidvishnoi marked this pull request as ready for review April 15, 2025 09:33
@sidvishnoi
sidvishnoi requested a review from AlexLakatos April 15, 2025 09:33
@sidvishnoi sidvishnoi added the specification Work required on specification label Jul 21, 2025
@uchibeke
uchibeke force-pushed the spec-walletAddress branch from ad906e1 to 79bf538 Compare July 19, 2026 08:26
@netlify

netlify Bot commented Jul 19, 2026

Copy link
Copy Markdown

Deploy Preview for webmonetization-preview failed. Why did it fail? →

Name Link
🔨 Latest commit 79bf538
🔍 Latest deploy log https://app.netlify.com/projects/webmonetization-preview/deploys/6a5c8a503a3b60000867d14c

@sublimator

Copy link
Copy Markdown
Collaborator

So this popped into my email, 2026, deprecating paymentPointer in favor of walletAddress

haha, my goodness :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

specification Work required on specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change paymentPointer attribute name to walletAddress

2 participants