feat!: remove all old deps by SgtPooki · Pull Request #360 · ipfs/ipld-explorer-components

SgtPooki · 2023-05-25T19:08:24Z

When will this PR be removed from Draft/WIP?

This PR was mainly intended to address #359, but in the process, we were able to solve a number of longstanding issues.

Preview of these changes in explore.ipld.io are available at https://bafybeici5vqsnnrggriwslm5m6ztoyedcumk2imy62ywwjficn5qt6fw6u.on.fleek.co/

This PR has the following changes:

Dev environment makes testing code changes in this repo really easy (via vite), instead of needing to always consume by explore.ipld.io or ipfs-webui in order to test changes
Creates and exports a helia bundle that is used instead of needing to define that bundle inside consuming packages
- The original intent to allow custom ipfs was sound, but has been changed for ease of maintenance. If users need to override, they can look at the bundle provided here and copy/modify as necessary in their appropriate implementations.
uses trustless gateway fetching, in a race with helia in src/lib/get-raw-block.ts (see

ipld-explorer-components/src/lib/get-raw-block.ts

Line 105 in 75461c2

const rawBlock = await Promise.any([helia.blockstore.get(cid, { signal: abortController.signal }), getBlockFromAnyGateway(cid, abortController.signal)])

)
uses helia to cache blocks (making loading MUCH faster)
- delegatedPeerRouting+delegatedContentRouting uses 'node3.delegate.ipfs.io' AND provided local node via kubo-rpc-client. Meaning helia manages fetching content when a gateway can't
removes react-scripts
removes old dependencies and updates all ipfs-ecosystem libraries to latest (/ipld|helia|ipfs/)

fixes #359
fixes #279
fixes #311
fixes #335
fixes #355

This is a large win for work required from ipfs/ipfs-webui#1965

related to explore.ipld.io changes at ipld/explore.ipld.io#119

Demo Video:

explore.ipld.io-2023-05-25-update.mp4

url is localhost:port/#/explore/bafybeiaysi4s6lnjev27ln5icwm6tueaw2vdykrtjkwiphwekaywqhcjze based on latest hash from https://github.com/ipfs/distributed-wikipedia-mirror/blob/1aa50702fc6d8e2be106e37d4c68dc2ca5b94f20/snapshot-hashes.yml#LL13C10-L13C69

fixes #355

… this package

This reverts commit ca0a50b.

whizzzkid

first pass looks good, any meaningful way to break this dowm?

.babelrc

.storybook/main.ts

whizzzkid · 2023-06-07T08:18:19Z

dev/devPage.jsx

+import 'react-virtualized/styles.css'
+import { composeBundles, createRouteBundle } from 'redux-bundler'
+import { Provider as ReduxStoreProvider, connect } from 'redux-bundler-react'
+import 'tachyons'


can we sort/organize imports across?

i'm not sure what that means. see .eslintrc.js 'import/order'.groups

dev/devPage.jsx

src/bundles/explore.js

whizzzkid · 2023-06-07T08:27:31Z

src/bundles/helia.ts

+  error: null
+}
+
+function getUserOpts (key: string): Record<string, unknown> {


why unknown?

Cause it can be anything.. it's localStorage

src/bundles/helia.ts

whizzzkid · 2023-06-07T08:32:02Z

src/components/cid-info/CidInfo.jsx

          <div className='red fw2 ma0 f7'>{cidErr.message}</div>
        </div>
-      )}
+          )}


this looks werid.

it's using standard style for formatting, so it should be correct. build runs lint which runs standard, so it would fail if it werent correct.

this is already a pretty large change, I'm trying not to modify any of the horrendous jsx or ternary statements that already existed before I got here. we can do a pass later

src/components/object-info/ObjectInfo.stories.jsx

whizzzkid

LGTM in principle, not seeing any major ⛳️

SgtPooki added 28 commits May 18, 2023 16:52

feat: migrate to vite

bdd5b1a

fix: polyfills and other stuff

192b78d

feat: parsing of root XKCD and 1 child node works

6ef54bd

feat: verify raw block, cache blocks in helia

b605881

fix: cleanup get-codec-for-cid & ensure eth support

227ef12

fix: graph rendering

d44c4d5

fix: verify blocks on the client side

b9fd9e3

fix: dag-pb node navigation

0c8c69e

feat: update start-exploring page with more examples

8d784bb

feat: add ipfs-geoip example

1dfa524

fixes #355

fix: dont split path in Path component

0669618

fix: typings

ea48ecc

feat: remove multibase and multihashes deps

f473f58

chore: cleanup explore bundle

2d9a7cc

chore: trying to get build working - get explore.ipld.io to work with…

15c1e06

… this package

chore: trying to bundle with rollup

577d09e

tmp: messsssyyyy

b7f44f7

fix: build works (mostly) in explore.ipld.io

24bdd9c

chore: separate vite more

4c46175

fix: fix loading in ipld-explorer-components

f11c1e2

tmp

0c6b597

chore: cjs-esm-react-scripts-hell

ca0a50b

Revert "chore: cjs-esm-react-scripts-hell"

2d3f441

This reverts commit ca0a50b.

fix: works in explore.ipld.io

7af56b9

fix: ethereum block and types

c2ddb4c

fix: ipld-eth dynamic imports

a8e2732

fix: dependency in explore.ipld.io

75461c2

SgtPooki mentioned this pull request May 25, 2023

feat: use [email protected] ipld/explore.ipld.io#119

Merged

2 tasks

chore: update github workflows node-version

09ab5e8

SgtPooki added 2 commits June 2, 2023 16:19

test: fix loading of dag-cbor example cheese path

a1bd438

chore: cleanup vite index.html

607889f

SgtPooki mentioned this pull request Jun 2, 2023

feat: merge with explore.ipld.io repo #362

Open

SgtPooki added 8 commits June 2, 2023 16:38

chore: revert locales loadPath change

5a261b9

chore: address all code cleanup PR comments

9948ac4

fix: remove blockcodec-to-ipld-format

72f6987

chore!: remove ipld-ethereum

eca2562

fix: access dynamic import via default prop

87bfac1

fix: storybook issue

1ce076a

docs: update readme with redux-bundler description

da70bfc

fix: change helia-bundle namespace

b50935e

SgtPooki changed the title ~~[WIP] feat!: remove all old deps~~ feat!: remove all old deps Jun 6, 2023

SgtPooki self-assigned this Jun 6, 2023

SgtPooki marked this pull request as ready for review June 6, 2023 19:50

SgtPooki requested a review from a team as a code owner June 6, 2023 19:50

SgtPooki requested review from lidel and whizzzkid June 6, 2023 19:50

SgtPooki mentioned this pull request Jun 6, 2023

feat!: use [email protected] ipfs/ipfs-webui#2117

Merged

3 tasks

fix: node style selection and multicodec isolation

42cb36a

whizzzkid reviewed Jun 7, 2023

View reviewed changes

SgtPooki added 4 commits June 7, 2023 13:31

fix: attempt to fetch block from kubo client directly

db31a53

fix: toHex function in extract-info

4ac823d

chore: address pr comments

a31c790

test: fix src/lib/resolve-ipld-path.test.js

82f3a4f

whizzzkid approved these changes Jun 8, 2023

View reviewed changes

SgtPooki merged commit 9fbaf6b into master Jun 8, 2023

SgtPooki deleted the remove-js-ipfs branch June 8, 2023 19:31

SgtPooki mentioned this pull request Jun 8, 2023

RC - 4.x #354

Closed

1 task

SgtPooki mentioned this pull request Jul 3, 2023

fix: issues with using in ESM webui #356

Closed

lidel mentioned this pull request Sep 20, 2024

ethereum blocks displayed incorrectly ipld/explore.ipld.io#32

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat!: remove all old deps#360

feat!: remove all old deps#360
SgtPooki merged 79 commits intomasterfrom
remove-js-ipfs

SgtPooki commented May 25, 2023 •

edited

Loading

Uh oh!

whizzzkid left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

whizzzkid Jun 7, 2023

Uh oh!

SgtPooki Jun 7, 2023

Uh oh!

Uh oh!

Uh oh!

whizzzkid Jun 7, 2023

Uh oh!

SgtPooki Jun 7, 2023

Uh oh!

Uh oh!

whizzzkid Jun 7, 2023

Uh oh!

SgtPooki Jun 7, 2023

Uh oh!

SgtPooki Jun 7, 2023

Uh oh!

Uh oh!

whizzzkid left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

SgtPooki commented May 25, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

When will this PR be removed from Draft/WIP?

Uh oh!

whizzzkid left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

whizzzkid Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

SgtPooki Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

whizzzkid Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

SgtPooki Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

whizzzkid Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

SgtPooki Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

SgtPooki Jun 7, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

whizzzkid left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

SgtPooki commented May 25, 2023 •

edited

Loading