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

Update to lastic-grant3.md #2272

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Update to lastic-grant3.md #2272

merged 2 commits into from
Apr 5, 2024

Conversation

poppyseedDev
Copy link
Contributor

@poppyseedDev poppyseedDev commented Apr 3, 2024

Revised Proposal Updates

Following the completion of our milestone, we encountered several unforeseen changes to our original proposal. Here are the key updates:

  • We have removed Pierina from the proposal.
  • We have decided against using a local version of the Rococo chain for indexing. Initially, we considered indexing from a locally run Rococo chain, but this approach significantly complicated the verification process. Furthermore, with the Rococo coretime chain operating effectively, the need for a locally run indexer is eliminated. This was only considered a necessary measure when the Rococo coretime chain was not functioning properly.
  • We have also decided against indexing the Rococo relay chain's information. This decision stems from realizing that the essential data can be effectively sourced from real-time data. Operating three indexers for the Rococo, Kusama, and Polkadot relay chains was deemed not financially viable when alternative methods are available.
  • Due to these changes, the budget required for the first milestone has been reduced by $3,000. Conversely, we've expanded the scope for the second milestone by $3,000. Instead of producing a basic mock-up code, we are now enhancing the full functionality of the presented pages.
  • Additional minor adjustments involve our data display approach. Initially, data obtained from Subscan did not meet our expectations in certain areas. To address this, we have supplemented it with data directly from the Polkadot JS API.

@github-actions github-actions bot added the admin-review This application requires a review from an admin. label Apr 3, 2024
@keeganquigley keeganquigley added amendment This PR proposes changes to an existing application. and removed admin-review This application requires a review from an admin. labels Apr 3, 2024
@keeganquigley keeganquigley self-assigned this Apr 3, 2024
keeganquigley
keeganquigley previously approved these changes Apr 3, 2024
Copy link
Contributor

@keeganquigley keeganquigley left a comment

Choose a reason for hiding this comment

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

Thanks @poppyseedDev by "grants" I think you mean "milestone" costs rather, but other than that LGTM.

@semuelle
Copy link
Member

semuelle commented Apr 3, 2024

  • Due to these changes, the budget required for the first grant has been reduced by $3,000. Conversely, we've expanded the scope for the second grant by $3,000. Instead of producing a basic mock-up code, we are now enhancing the full functionality of the presented pages.

Just to clarify, this is referring to milestones 1 and 2, not grants.

Noc2
Noc2 previously approved these changes Apr 4, 2024
Copy link
Member

@PieWol PieWol left a comment

Choose a reason for hiding this comment

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

generally looks good to me.

applications/lastic-grant3.md Outdated Show resolved Hide resolved
@poppyseedDev poppyseedDev dismissed stale reviews from Noc2 and keeganquigley via a260cd7 April 4, 2024 12:03
@github-actions github-actions bot added the admin-review This application requires a review from an admin. label Apr 4, 2024
@poppyseedDev
Copy link
Contributor Author

Thanks @poppyseedDev by "grants" I think you mean "milestone" costs rather, but other than that LGTM.

  • Due to these changes, the budget required for the first grant has been reduced by $3,000. Conversely, we've expanded the scope for the second grant by $3,000. Instead of producing a basic mock-up code, we are now enhancing the full functionality of the presented pages.

Just to clarify, this is referring to milestones 1 and 2, not grants.

Yes correct, thank you for the clarification, I fixed the wording.

@keeganquigley keeganquigley removed the admin-review This application requires a review from an admin. label Apr 4, 2024
@semuelle semuelle merged commit a3f4a37 into w3f:master Apr 5, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amendment This PR proposes changes to an existing application.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants