perf: return cf-images #302

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

Draft

preschian wants to merge 1 commit into main from perf--return-cf-images

Member

preschian commented Jun 11, 2024

Context

describe why do we need this change?
what is the problem?
how did you solve the problem?

Ref

Closes #<issue_number>
References kodadot/<indexer/worker>#<issue_number>

PR Type

Bugfix
Feature
Refactoring
Chore


          perf: return cf-images

00ee151

vikiival reviewed

View reviewed changes

services/image/src/utils/ipfs.ts

    
                const gateways: HTTPS_URI[] = [

                  ipfsProviders.ipfs,

                  ipfsProviders.dweb,

Member

vikiival Jun 23, 2024

Suggested change

      
                ipfsProviders.dweb,
          
                ipfsProviders.w3s,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

vikiival vikiival left review comments

hassnian Awaiting requested review from hassnian hassnian will be requested when the pull request is marked ready for review hassnian is a code owner

Jarsen136 Awaiting requested review from Jarsen136 Jarsen136 will be requested when the pull request is marked ready for review Jarsen136 is a code owner

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

Labels

None yet