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

Added Node Red project #2125

Closed
wants to merge 8 commits into from
Closed

Added Node Red project #2125

wants to merge 8 commits into from

Conversation

johnuopini
Copy link

Project Abstract

The Node Red Client for Polkadot by Zooper Corp is a practical tool designed to simplify the use of blockchain technology. It integrates the Polkadot ecosystem with Node Red, a user-friendly flow-based programming interface. This project is focused on making blockchain interactions straightforward and accessible, especially for those without extensive technical knowledge. Users can track transactions, automate processes, and interact with the Polkadot network more easily. This application is our first request for support under the Web3 Foundation's Maintenance Grants Program and represents an independent initiative developed by Zooper Corp.

Grant level

  • Level 1: Up to $10,000, 2 approvals
  • Level 2: Up to $30,000, 3 approvals
  • Level 3: Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

  • The application template has been copied and aptly renamed (project_name.md).
  • I have read the application guidelines.
  • Payment details have been provided (bank details via email or Polkadot (USDC & USDT) address in the application).
  • The software delivered for this grant will be released under an open-source license specified in the application.
  • The initial PR contains only one commit (squash and force-push if needed).
  • The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
  • I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @johnuopini:matrix.org (change the homeserver if you use a different one)

Copy link
Contributor

github-actions bot commented Nov 24, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@johnuopini
Copy link
Author

I have read and hereby sign the Contributor License Agreement.

Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

Thanks for the application. Could you change the DOT payment to one of the other supported payment options: https://github.com/w3f/Grants-Program#pencil-process? Apart from this, could you add more details to the milestone deliveries, especially regarding the second milestone? The milestones are basically the requirements of our contracts that we need to evaluate. Milestone 2 currently is completely unspecific and could mean anything. It might make even sense to remove it completely for now and later apply for another grant or treasury funding.

@Noc2 Noc2 added details missing Not enough technical details. changes requested The team needs to clarify a few things first. labels Nov 24, 2023
Copy link
Contributor

Thank you for submitting a grant application.

We've assessed your submission and have found that it requires a higher level of technical detail in order to be considered for review. We encourage you to expand on it by providing a more precise specification/technical details. The section on project details in the application template is a good reference as to what type of information we expect applicants to provide, and these category-specific requirements contain more precise guidelines depending on what type of software you're building.

An area of the application that we often find to be insufficiently elaborated are the milestone deliverables. At a minimum, please indicate what languages/technologies you will be using to implement each deliverable, and provide a technical summary of its expected functionality. Note that deliverables should be tangible, reusable by other teams and in most cases not already present in the ecosystem. If they are, you will need to provide a comparison to existing implementations and explain why it makes sense to fund your approach. Also see our FAQ for a breakdown of what we fund and what we don't.

Let us know as soon as you're done with your changes, and we'll give your application another look!

@semuelle semuelle self-assigned this Nov 27, 2023
@semuelle semuelle added the discussion private Discussion of application happens in private. label Nov 27, 2023
Copy link
Contributor

github-actions bot commented Nov 27, 2023

The applicant has requested the discussion of the application to happen in a private chat room.

- Added detailed milestones
- Added DAI address and costs
@johnuopini
Copy link
Author

I have addressed following points:

  • Added a DAI address
  • Added detailed milestones on how the first maintenance year might look like, this is also a maintenance request so the idea is in parallel to also keep fixing bugs and checking for pull requests from other users

@johnuopini johnuopini requested a review from Noc2 November 27, 2023 18:01
Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

Thanks for the update. However, each milestone should have the default deliverables of the template see https://github.com/w3f/Grants-Program/blob/master/applications/application-template.md#milestone-1-example--basic-functionality. Because of this, I also recommend to combine milestone 1 and 2. We usually want to have inline documentation to be able to test the delivery. Furthermore, my recommendation would be to initially only apply for a regular grant and apply later for the maintenance part. For example, "Addressing and resolving issues " could mean anything at this stage.

- Merged milestones
- Consolidate documentation / test tasks into one
- Made milestones compliant with template
@johnuopini
Copy link
Author

  • Merged milestones
  • Consolidate documentation / test tasks into one
  • Made milestones compliant with template

@johnuopini johnuopini requested a review from Noc2 November 28, 2023 10:18
@semuelle semuelle added the admin-review This application requires a review from an admin. label Nov 29, 2023
Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

Thanks for the update. However, it's still not very specific. For example: " Web3 technologies" could mean anything. The same is true for "basic Node Red nodes". Additionally, all milestones should contain the default deliverables of the template (0.a to 0d.). Could you try to improve the application and add more details?

- Clarified stack
- Added docker to default deliverables
- Clarified objectives
@johnuopini
Copy link
Author

Thanks for the feedback, doc was updated:

  • Clarified stack
  • Added docker to default deliverables
  • Clarified objectives

@johnuopini johnuopini requested a review from Noc2 December 4, 2023 16:48
Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

Thanks for the update. I will mark the application as ready for review. However, I'm personally not convinced by the project and the usability. Also, the default deliverables are still missing for milestones 2, 3, and 4! Furthermore, I don't think the Maintenance Grant should be part of this proposal. But maybe the rest of the committee has a different opinion.

@Noc2 Noc2 added ready for review The project is ready to be reviewed by the committee members. and removed changes requested The team needs to clarify a few things first. details missing Not enough technical details. labels Dec 5, 2023
- Added default steps in milestones 2/3/4
- Clarified maintenance
@johnuopini
Copy link
Author

  • Added default deliverables
  • Clarified maintenance

@keeganquigley
Copy link
Contributor

keeganquigley commented Dec 5, 2023

Thanks @johnuopini for all your time and effort in making the changes to the application. I must agree though, that I'm also not convinced about the usability of the project. Node Red is cool, but it seems that you'd need to use a proxy in order to send txs which complicates things, and I'm not sure how this is any easier than learning how to use user-friendly wallets such as Talisman, or Multix for multi-sig txs.

In my opinion, anyone making advanced or complicated txs should probably learn what they are doing anyway, since many things can go wrong. We've already had many users get funds stuck because they didn't understand how anonymous proxies work, for example.

I also see the the maintenance grant is still included, which would have to be removed and funded under a separate maintenance grant. Also we don't pay in DOT, so the payment option would need to be changed there anyway.

For these reasons as well as the ones above, we've decided to go ahead and close this application. But I thank you again for your time, and wish you the best of luck in finding funding.

@johnuopini
Copy link
Author

@keeganquigley thanks for looking into this, you do not actually need a proxy, nodered has plugins for vaults that can be used to pull secrets and i could include a way to securely store a seed. Also i am not sure you understood how Node Red works, Node Red is a visual "no code" programming platform and can be hosted on docker, k8s or basically anything, having a polkadot client there allow users to quickly develop polkadot flows like they would on IFTTT, for example you could quickly create a telegram bot that alerts you whenever your balance hits a given amount or a specific event occurs, honestly i do not see any visual way to do that today, how would you implement such a logic in Talisman or Multix?? Node Red is a bout automation and no code programming, it has nothing to do with a wallet so maybe the use case was not understood

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin-review This application requires a review from an admin. discussion private Discussion of application happens in private. ready for review The project is ready to be reviewed by the committee members.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants