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.

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

I can has extension? #623

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@ZmnSCPxj
Copy link
Collaborator

commented Jun 17, 2019

Proposal for permissionless development of experimental features.

I believe this can be added to the spec almost entirely as-is, although it does remove 25% of the possible space of available messages for itself.

@ZmnSCPxj ZmnSCPxj force-pushed the ZmnSCPxj:icanhasextension branch from cbc39e1 to 6d4b70b Jun 17, 2019

@renepickhardt
Copy link
Contributor

left a comment

I think this idea is pretty useful. I passed my feedback inline. I think we should really change the name of the messages. My other feedback seems like it is all already in there but I thought I would still drop it as it might improve the overall readability of this BOLT because these where the points while reading with which I seemed to have some issues.

Show resolved Hide resolved 13-i-can-has-extension.md Outdated
Show resolved Hide resolved 13-i-can-has-extension.md
Show resolved Hide resolved 13-i-can-has-extension.md
Show resolved Hide resolved 13-i-can-has-extension.md
Show resolved Hide resolved 13-i-can-has-extension.md

@ZmnSCPxj ZmnSCPxj force-pushed the ZmnSCPxj:icanhasextension branch from 6d4b70b to 5337c7b Jul 1, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.