forgejo-admin/nuts
0
0
Fork 0
Code Issues 35 Pull requests 32 Projects Releases Packages Wiki Activity Actions

NUT-XX: get locked mint quotes by public keys #329

Closed
average-gary wants to merge 2 commits from main into main
pull from: main
merge into: forgejo-admin:main
Conversation 6 Commits 2 Files changed 4 +130
average-gary commented 2026-01-12 20:55:23 +00:00 (Migrated from github.com)
Copy link

The primary use case I have in mind is ehash mining but examples use bolt11 LN for simplicity and broader knowledge of that flow for Cash protocol engineers.

The primary use case I have in mind is ehash mining but examples use bolt11 LN for simplicity and broader knowledge of that flow for Cash protocol engineers.
thesimplekid (Migrated from github.com) reviewed 2026-01-12 20:55:23 +00:00
thesimplekid (Migrated from github.com) requested changes 2026-01-13 09:33:38 +00:00
thesimplekid (Migrated from github.com) left a comment
Copy link

This proposal could be stripped down to just adding one endpoint that returns quotes for a given pubkey. The rest stays as already defined in other specs.

This proposal could be stripped down to just adding one endpoint that returns quotes for a given pubkey. The rest stays as already defined in other specs.
28.md
@ -1,176 +1,93 @@
# NUT-28: Pay-to-Blinded-Key (P2BK)
# NUT-28: Mint Quote Lookup by Public Key
thesimplekid (Migrated from github.com) commented 2026-01-13 09:25:56 +00:00
Copy link

This is the same as nut20 currently is, the mint does not need to know that its intended for a 3rd party.

This is the same as nut20 currently is, the mint does not need to know that its intended for a 3rd party.
thesimplekid (Migrated from github.com) commented 2026-01-13 09:28:07 +00:00
Copy link

This is the same as current melt flow I don't think we need to redefine it.

This is the same as current melt flow I don't think we need to redefine it.
thesimplekid (Migrated from github.com) commented 2026-01-13 09:28:46 +00:00
Copy link

Don't think this should be in this spec either.

Don't think this should be in this spec either.
thesimplekid (Migrated from github.com) commented 2026-01-13 09:29:33 +00:00
Copy link

We don't delete these.

We don't delete these.
thesimplekid (Migrated from github.com) commented 2026-01-13 09:31:19 +00:00
Copy link

This also seems like an application specific thing and not for the nut. Also should not use nut04

This also seems like an application specific thing and not for the nut. Also should not use nut04
average-gary commented 2026-01-13 19:28:58 +00:00 (Migrated from github.com)
Copy link

@thesimplekid would it be preferred to modify NUT20 since it's just adding an endpoint?

@thesimplekid would it be preferred to modify NUT20 since it's just adding an endpoint?
thesimplekid commented 2026-01-13 23:47:36 +00:00 (Migrated from github.com)
Copy link

@thesimplekid would it be preferred to modify NUT20 since it's just adding an endpoint?

No, I don't think we should try and change an existing nut for new functionality.

> @thesimplekid would it be preferred to modify NUT20 since it's just adding an endpoint? No, I don't think we should try and change an existing nut for new functionality.
callebtc commented 2026-02-16 12:27:17 +00:00 (Migrated from github.com)
Copy link

I think this is needed but the NUT should be shortened significantly. I'll suggest some edits.

I think this is needed but the NUT should be shortened significantly. I'll suggest some edits.
👍 1
callebtc commented 2026-02-16 12:38:54 +00:00 (Migrated from github.com)
Copy link

Superseded by #341 because this PR doesn't allow maintainer changes.

Superseded by #341 because this PR doesn't allow maintainer changes.

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
thesimplekid
Labels
Clear labels   
breaking change
This issue/PR entails breaking changes

  
bug
Something isn't working

  
documentation
Improvements or additions to documentation

  
enhancement
New feature or request

  
needs discussion
Needs more discussion

  
needs implementation
Needs a reference implementation

  
new nut
A new protocol NUT

  
ready
Ready to merge

  
wallet-only
A change that affects only wallets

No labels
breaking change
bug
documentation
enhancement
needs discussion
needs implementation
new nut
ready
wallet-only
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo-admin/nuts!329
No description provided.