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.

Sign up for GitHub

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

Jump to bottom

Feature: Indexing new contracts #24

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Feature: Indexing new contracts #24

wants to merge 31 commits into from

Conversation

brunomenezes
Copy link
Collaborator

@brunomenezes brunomenezes commented Jan 23, 2024 •
edited
Loading

Summary

Code changes to add index data from contracts and events of interest.

More info:

closes #23

@brunomenezes brunomenezes requested a review from tuler January 23, 2024 05:01
@brunomenezes brunomenezes linked an issue Jan 23, 2024 that may be closed by this pull request
Indexing new providers contracts #23
Open
7 tasks
@tuler
Copy link
Member

tuler commented Jan 23, 2024

Looking good so far

@brunomenezes
feat: Add new graphQL entities
10332d1
@brunomenezes
chore: Bump dependencies and update vitest config
c471b14
@brunomenezes
refactor: Replace deprecated processor config
b0f899a
@brunomenezes
feat: Add Authority handler.
e0fcab4
@brunomenezes
feat: Update config and processor setup.
287278b
@brunomenezes
feat: Add new migration
5c1d8ff
@brunomenezes
feat: Add ValidatorNodeProvider event handlers + helper.
ac7c37b
@brunomenezes
feat: Add ValidatorNode event handlers
00e72a7
@brunomenezes
feat: Add new handlers and maps into EventHandler.
bcc1f5f
@brunomenezes
feat: Save new entities and include total and summary refactoring
2cc0a32
@brunomenezes
feat: Add support to Arbitrum and Arbitrum-Goerli
b21cc1e
@brunomenezes
chore: Bump application preloaders for Sepolia and Mainnet
bb5efa3
@brunomenezes
feat: Add preload files for Arbitrum and Arbitrum-Goerli
57e4997
@brunomenezes
feat: Add preloaders for validator node providers including npm-scrip…
664c9e9 
…t and new command definition
@brunomenezes
refactor: Replace deprecated processor setting on Application loader …
1e52e1c 
…script.
@brunomenezes
refactor: Change inputAdded to use TokenHelper
a590193
@brunomenezes
test: Add test cases for ValidatorNodeProvider + Stubs.
a617d06
@brunomenezes
test: Add test cases for Paused and Unpaused provider events.
2fa6b71
@brunomenezes
test: Add test cases for ValidatorNodeProviderCreated events.
42d9641
@brunomenezes
test: Add cases for ValidatorNodeFinancialRunway events
cd44178
@brunomenezes
test: Add cases for ValidatorNodeMachineLocation event
82560e9
@brunomenezes brunomenezes force-pushed the feature/23-indexing-new-providers-contracts branch from 205b9fe to 82560e9 Compare January 26, 2024 07:36
@brunomenezes
Copy link
Collaborator Author

brunomenezes commented Jan 29, 2024 •
edited
Loading

@tuler, I am changing the PR to ready to review, but there are a few things to notice. Even though the diagram has some arrows between ValidatorNodeProvider to Marketplace and also to ApplicationFactory these are actually out because the factory is the Marketplace and the ApplicationFactory is searchable once you have the ValidatorNode, so the final Diagram would be as below. Let me know if the way to reach ApplicationFactory needs to be through the provider and not the ValidatorNode itself.

full_diagram

@brunomenezes brunomenezes marked this pull request as ready for review January 29, 2024 07:33
tuler
tuler reviewed Feb 1, 2024
View reviewed changes
src/config.ts Outdated Show resolved Hide resolved
src/config.ts Outdated Show resolved Hide resolved
@brunomenezes brunomenezes force-pushed the feature/23-indexing-new-providers-contracts branch from eb3c481 to 887616c Compare February 1, 2024 23:56
tuler
tuler reviewed Feb 2, 2024
View reviewed changes
Copy link
Member

@tuler tuler left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I think I'll need to navigate from application to the collection of validatorNodes

@brunomenezes brunomenezes mentioned this pull request Feb 2, 2024
Merged
@brunomenezes
Copy link
Collaborator Author

I think I'll need to navigate from application to the collection of validatorNodes

I merged the PR #25 here that accounts for that and simplify the Entities as we discussed previously.

@brunomenezes brunomenezes self-assigned this Feb 7, 2024
@brunomenezes brunomenezes added the Status: On-Hold On-hold for further development, because upcoming changes will affect the Issue or PR targeted label Feb 7, 2024
@brunomenezes brunomenezes mentioned this pull request Feb 18, 2024
Merged
@tuler tuler mentioned this pull request Apr 30, 2024
Closed
@brunomenezes brunomenezes reopened this Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tuler tuler tuler left review comments

@nevendyulgerov nevendyulgerov Awaiting requested review from nevendyulgerov

@dandheedge dandheedge Awaiting requested review from dandheedge

At least 1 approving review is required to merge this pull request.

Assignees

@brunomenezes brunomenezes

Labels
Status: On-Hold On-hold for further development, because upcoming changes will affect the Issue or PR targeted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Indexing new providers contracts
2 participants
@brunomenezes @tuler