Skip to content

Conversation

@SgtPooki
Copy link
Contributor

@SgtPooki SgtPooki commented Sep 22, 2025

  • updated all deps, then parcel wouldnt build.
  • migrated to vite
  • updated codecs with existing script
  • removed unnecessary node-fetch
  • removed old peer-id dep and replaced with @libp2p/peer-id

And added retrieval check after successful CID parsing

image

This is the final task for ipshipyard/roadmaps#11

updated all deps, then parcel wouldnt build.
migrated to vite
updated codecs with existing script
removed unnecessary node-fetch
@github-actions
Copy link

github-actions bot commented Sep 22, 2025

🚀 Build Preview on IPFS ready

@SgtPooki SgtPooki requested a review from lidel September 22, 2025 17:29
Copy link
Contributor Author

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

self review

- move retrieval check to bottom for better information hierarchy
- add encodeURIComponent for URL parameter (defensive coding)
- fix spacing issue when DNS section hidden (display:none vs visibility:hidden)
- improve phrasing to be more user-friendly
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

Thank you for upgrading this project.

I've moved retrieval check to the bottom to flow better with the purpose of this tool (inspect CID first, other representations and diagnostics second)

@lidel lidel merged commit 8956377 into main Sep 23, 2025
3 checks passed
@lidel lidel deleted the feat/check-ipfs-link branch September 23, 2025 02:19
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.

3 participants