-
Notifications
You must be signed in to change notification settings - Fork 196
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
feat(explorer): dozer integration #3185
Conversation
🦋 Changeset detectedLatest commit: 6b8a21a The changes in this PR will be included in the next version bump. This PR includes changesets to release 26 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Co-authored-by: alvarius <[email protected]>
packages/explorer/src/app/(explorer)/queries/useDeployedTablesQuery.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Kevin Ingersoll <[email protected]>
.../explorer/src/app/(explorer)/[chainName]/worlds/[worldAddress]/explore/EditableTableCell.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
I personally prefer the table
naming of Table
objects rather than tableConfig
as it matches the rest of MUD (minus stash)
Key changes:
DataExplorerSqlite
andDataExplorerDozer
to distinguish which indexer is used under-the-hood.TablesViewer
andTableSelector
interfaces so they work for both,DataExplorerSqlite
andDataExplorerDozer
.SQLEditor
that is included as part ofDataExplorerDozer
.tables
,rows
and other routes. Only left withapi/world
for fetching world ABI, andapi/table
to fetch a specific data table (when using sqlite indexer).queries/**
helpers.store__tables
(previously it was fetched from all available SQL databases using its pragma).Demo world - http://localhost:13690/garnet/worlds/0x9ca094a38027bb331656e4f1f6ed40c9982dbe70/explore
todo: