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

Sdk-dapp-core migration #264

Open
wants to merge 137 commits into
base: development
Choose a base branch
from
Open

Conversation

razvantomegea
Copy link
Contributor

No description provided.

import {
apiTimeout,
environment,
sampleAuthenticatedDomains,
Copy link
Contributor

@arhtudormorar arhtudormorar Feb 17, 2025

Choose a reason for hiding this comment

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

bring back or use in promise.all @razvantomegea

apiTimeout,
environment,
sampleAuthenticatedDomains,
walletConnectV2ProjectId
Copy link
Contributor

@arhtudormorar arhtudormorar Feb 17, 2025

Choose a reason for hiding this comment

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

use this walletConnectV2ProjectId @razvantomegea

vite.config.ts Outdated
import { defineConfig } from 'vite';
import { nodePolyfills } from 'vite-plugin-node-polyfills';
import svgrPlugin from 'vite-plugin-svgr';
import tsconfigPaths from 'vite-tsconfig-paths';

export default defineConfig({
//TODO remove this when excluded from optional dependencies
optimizeDeps: {
exclude: ['@multiversx/sdk-dapp-core-ui'] // Prevent Vite from trying to pre-bundle it
Copy link
Contributor

Choose a reason for hiding this comment

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

remove @DanutIlie ?

* Added unlock panel

* resolve comments

* close panel after login
anchor
});
await provider?.login();
setOpen(false);
Copy link
Contributor

Choose a reason for hiding this comment

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

check

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