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

Update subgraph to use TheGraph #14

Merged
merged 5 commits into from
Oct 3, 2024
Merged

Update subgraph to use TheGraph #14

merged 5 commits into from
Oct 3, 2024

Conversation

boyuan-chen
Copy link
Contributor

@boyuan-chen boyuan-chen commented Oct 2, 2024 •
edited by sk-enya
Loading

Deploy all subgraphs to TheGraph Studio

// change by sahil:

  • Setup jest to read the env param
  • Update the subgraph url
  • Added spec for withdrawalInitiated, withdrawalInitiatedBnb, MessagePassed.
  • Updated spec for anchorage service
    Screenshot 2024-10-03 at 10 14 42 AM

@boyuan-chen boyuan-chen requested review from wsdt and sk-enya October 2, 2024 19:51
@boyuan-chen
Copy link
Contributor Author

@wsdt @sk-enya what is the right way to test it?

@sk-enya
Copy link
Contributor

sk-enya commented Oct 3, 2024

I used to test it by linking this module to gateway project locally earlier but it seems to have spec corresponding to this in same repo, let me get the quick spec setup for this service!

sk-enya added 3 commits October 3, 2024 09:43
@sk-enya
change:
7bb59ca 
- setup jest to read the env param
- update the subgraph url
- added spec for withdrawalInitiated, withdrawalInitiatedBnb, MessagePassed.
@sk-enya
updated name
e70aca1
@sk-enya
updated spec for anchorage service
d4022b3
wsdt
wsdt approved these changes Oct 3, 2024
View reviewed changes
Copy link
Contributor

@wsdt wsdt left a comment

Choose a reason for hiding this comment

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

nice!

@boyuan-chen boyuan-chen merged commit af364bd into main Oct 3, 2024
2 checks passed
@boyuan-chen boyuan-chen deleted the update-subgraph branch October 3, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sk-enya sk-enya sk-enya approved these changes

@wsdt wsdt wsdt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@boyuan-chen @sk-enya @wsdt