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

[hats-protocol-hat-ids] Fix hats-protocol-hat-ids strategy #1368

Merged
merged 1 commit into from
Dec 4, 2023
Merged

[hats-protocol-hat-ids] Fix hats-protocol-hat-ids strategy #1368

merged 1 commit into from
Dec 4, 2023

Conversation

gershido
Copy link
Contributor

@gershido gershido commented Dec 4, 2023

Fixes hats-protocol-hat-ids strategy.

Changes proposed in this pull request:

  • Fix hat ID conversion
  • Remove subgraph usage

@ChaituVR ChaituVR changed the title Fix hats-protocol-hat-ids strategy [hats-protocol-hat-ids] Fix hats-protocol-hat-ids strategy Dec 4, 2023
@ChaituVR
Copy link
Member

ChaituVR commented Dec 4, 2023

@hotmanics Could you look into this also and confirm? thanks

@JacobHomanics
Copy link
Contributor

@Hotmanics Could you look into this also and confirm? thanks

I am confirming these changes!

@ChaituVR ChaituVR merged commit 9603642 into snapshot-labs:master Dec 4, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChaituVR ChaituVR ChaituVR approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@gershido @ChaituVR @JacobHomanics