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

feat: Start working on a simple cross window RPC provider #166

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

richtera
Copy link
Contributor

@richtera richtera commented Oct 7, 2024

feat: Add starter project for RPC provider and hook it into Home.vue for testing.

@richtera richtera force-pushed the feat/embedded-provider branch 2 times, most recently from d289d18 to 1081c29 Compare October 8, 2024 16:28
Copy link

github-actions bot commented Oct 8, 2024

Deployed with Cloudflare Pages ☁️ 🚀 🆗

Copy link
Contributor

@dzbo dzbo left a comment

Choose a reason for hiding this comment

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

Maybe let's separate line width update from the PR, add a lot of additional changes and makes review harder

@richtera richtera merged commit adf4db5 into main Nov 1, 2024
1 check passed
@richtera richtera deleted the feat/embedded-provider branch November 1, 2024 14:26
@@ -10,3 +10,5 @@
!.*
.eslintcache
/.vscode

/packages/tools-lukso-provider/docs/**/*.md
Copy link
Member

Choose a reason for hiding this comment

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

Why don't we ignore the whole packages directory?

DeployedLSP8IdentifiableDigitalAsset,
LSP7DigitalAssetDeploymentOptions,
LSP8IdentifiableDigitalAssetDeploymentOptions,
} from '@lukso/lsp-factory.js/build/main/src/lib/interfaces/digital-asset-deployment'
Copy link
Member

Choose a reason for hiding this comment

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

Is this the factory that we stopped using a long time ago, or is this a different package? Also, the import looks a bit strange 🤔

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

Successfully merging this pull request may close these issues.

3 participants