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.

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

Update Infura IPFS Provider API with Pinata IPFS Provider API for I… #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

upendra-eth
Copy link

…PFS integration

Description

This pull request updates the IPFS integration in the NFT Marketplace project by switching from the Infura API to the Pinata API. Infura recently deprecated their public gateway, requiring users to create an account and provide card details to access the free API. However, this additional step of providing card details is often the last thing students and developers would prefer to do. To simplify the process and make it more convenient, I have replaced the Infura API calls with equivalent calls to the Pinata API, which does not require card details. This change ensures smooth functionality for IPFS-related operations in the application while addressing the concerns of students and developers.

Changes Made

  • Updated the ipfs.js file to replace all Infura API endpoints with corresponding Pinata API endpoints.
  • Updated the config.js file to use the Pinata API key and secret for authentication.
  • Refactored relevant code sections to accommodate changes in the API response format.
  • Added the required Pinata SDK and dependencies to the project's package.json file.
  • Updated the README.md file with instructions on how to obtain a Pinata API key and configure it for use in the application.

Testing Done

  • Conducted thorough testing of the IPFS integration using Pinata API.
  • Verified that file uploads, pinning, and retrieval of IPFS content function as expected.
  • Tested the application's various features related to IPFS, such as NFT creation, metadata storage, and retrieval.

Related Issues

5 - IPFS image upload error
9 - ipfs image upload error: HTTPError: project id required

Closes #5 #9

Checklist

  • Updated the code to use Pinata API instead of Infura API.
  • Conducted testing and verified the functionality.
  • Updated the necessary documentation.
  • Ensured backward compatibility with existing features.

@dweezymonae
Copy link

i was encountering the same issue, unfortunately your PR was not merged - is there a way to merge this update into my local branch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IPFS image upload error
4 participants