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

Create substrate-graphql-api.md #2421

Closed
wants to merge 1 commit into from
Closed

Conversation

justmert
Copy link
Contributor

Project Abstract

GraphQL API for seamless Substrate blockchain interactions.

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 (Polkadot AssetHub (USDC & DOT) address in the application and bank details via email, if applicable).
  • I understand that an agreed upon percentage of each milestone will be paid in vested DOT, to the Polkadot address listed in the application.
  • I am aware that, in order to receive a grant, I (and the entity I represent) have to successfully complete a KYC/KYB check.
  • 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: @_______:matrix.org (change the homeserver if you use a different one)

@github-actions github-actions bot added the admin-review This application requires a review from an admin. label Oct 11, 2024
Copy link
Contributor

github-actions bot commented Oct 11, 2024

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

@justmert
Copy link
Contributor Author

I have read and hereby sign the Contributor License Agreement.

@PieWol PieWol self-assigned this Oct 14, 2024
@PieWol
Copy link
Member

PieWol commented Oct 14, 2024

Hey @justmert ,
thank you for applying to our Grants Program :)

On your LinkedIn it looks like you already have a full time position. Are you willing to work two FTE's simultaneously or is this project that you are trying to fund for your company ApeWorx?

After a few minutes of research I was able to find SubQuery and I'm surprised you didn't include it in the section of "comparable projects" in your application. To me it looks like SubQuery is already offering what you are proposing here. I would love to hear your thoughts on how you compare to them.

@justmert
Copy link
Contributor Author

Hey @PieWol, thank you for your quick review :)

Actually this project is not affiliated with Apeworx. I am not employed by Apeworx; our arrangement is a contract-based agreement for the delivery of a specific project. Right now, we're only handling maintenance, so it's not a full-time arrangement. For this new project, however, I'll be working full-time.

Sorry to not include SubQuery, I was aware of it but I made direct comparison between project's direct counterpart substrate-rest-api project. Here's the comparison between this project and SubQuery:

  1. Real-time vs Indexed Data:
  • This project provides data directly from a Substrate node.
  • SubQuery indexes blockchain data and serves it from a database.
  • Real-time data is useful for current information, while indexed data suits historical analysis.
  1. Setup and Configuration:
  • This project offers setup with a configuration UI for quick developer onboarding.
  • SubQuery requires setting up an indexing project, allowing more customization but with added complexity.
  1. Customization vs Standardization:
  • This project provides a standard API for common Substrate chain interactions.
  • SubQuery allows custom data models and queries, beneficial for specific project needs.
  1. Resource Requirements:
  • This project has lower resource needs without a separate database.
  • SubQuery requires more resources for indexing but offers faster queries for large historical datasets.
  1. Use Case Focus:
  • This project suits applications needing current Substrate data.
  • SubQuery is better for apps requiring historical data analysis or large data volume queries.
  1. Learning Curve:
  • Developers familiar with GraphQL can use this project quickly.
  • SubQuery requires learning about indexing and custom mappings.

SubQuery is a great project but actually differs from this project a lot. We best compare this project with its counterpart substrate-api-sidecar which is a REST API service for Substrate nodes.

@PieWol
Copy link
Member

PieWol commented Oct 14, 2024

Got it, thanks for clarifying.

@PieWol PieWol added the ready for review The project is ready to be reviewed by the committee members. label Oct 14, 2024
@PieWol
Copy link
Member

PieWol commented Oct 25, 2024

Hey @justmert ,
with your latest clarifications it does seem like there is nothing currently in the space which uses GraphQL to directly query nodes instead of indexers. I can't seem to find any evidence that this is actually needed and desired within the ecosystem. Given that your two previous grants (Dotly and PolkaFlow) became immediately stale once they were approved by W3F, I'm not hopeful that this very project will end any different. I'm sorry but just by these few but strong points I'm not fond of approving it. I'm open to revisit this in case you manage to provide evidence of demand for this tool. Maybe my colleagues have a different view on your application and I hope that they will provide some feedback or their decision soon. I already shared the application with the committee. Until then best wishes 🙏

@PieWol
Copy link
Member

PieWol commented Oct 28, 2024

Hey @justmert ,
due to the aforementioned reasons the committee sadly wasn't convinced by your application. I will close this PR now. We appreciate the time and effort you put into this application.

Best of luck on your future endeavours. Feel free to reach out if you have any questions.

@PieWol PieWol closed this Oct 28, 2024
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. 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.

2 participants