Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

fix(subgraph): stop to use ipfs in subgraph #126

Closed
wants to merge 2 commits into from
Closed

fix(subgraph): stop to use ipfs in subgraph #126

wants to merge 2 commits into from

Conversation

julien-devatom
Copy link
Contributor

Description of change

  • from now, we are uploading a new distribution to ipfs and rerunning the subgraph. However, the pinned distribution is not always indexed by all ipfs node, and subgraph is just failing.

The solution is just to use a Mapping instead of using a ipfs file

Pull-Request Checklist

  • Code is up-to-date with the main branch
  • yarn lint passes with this change
  • yarn test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits follow conventions outlined in the conventional commit spec

@julien-devatom julien-devatom self-assigned this Aug 20, 2023
@julien-devatom julien-devatom requested review from a team, ghivert, mlgarchery and oumar-fall and removed request for a team August 20, 2023 15:54
@julien-devatom julien-devatom marked this pull request as draft August 21, 2023 06:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ghivert ghivert Awaiting requested review from ghivert ghivert was automatically assigned from morpho-org/interface-team

@mlgarchery mlgarchery Awaiting requested review from mlgarchery mlgarchery was automatically assigned from morpho-org/interface-team

@oumar-fall oumar-fall Awaiting requested review from oumar-fall oumar-fall was automatically assigned from morpho-org/interface-team

Assignees

@julien-devatom julien-devatom

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@julien-devatom