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

fix Kibisis init in vue app #203 #225

Merged
merged 3 commits into from
Sep 13, 2024
Merged

fix Kibisis init in vue app #203 #225

merged 3 commits into from
Sep 13, 2024

Conversation

scholtz
Copy link
Contributor

@scholtz scholtz commented Aug 30, 2024

i faced the same issue as in #203 the issue is that the import in vuejs has the default, so fixed it the same way as it is done in defly f.e.

i faced the same issue as in TxnLab#203 the issue is that the import in vuejs has the default, so fixed it the same way as it is done in defly f.e.
@kieranroneill
Copy link
Contributor

Looking good to me 👍

So this relates to AVM Web Provider using the old style CommonJS and UseWallet using ESM?

@drichar
Copy link
Collaborator

drichar commented Aug 30, 2024

Thanks for this fix! Please run the linting/formatting scripts in the root of the repository and fix any reported errors. I'll need to add a CONTRIBUTING.md file with guidelines and instructions.

@scholtz
Copy link
Contributor Author

scholtz commented Aug 31, 2024

i have just updated the code in github web, feel free to run the linting after you merge it or setup some pipeline to do so automatically

@drichar drichar merged commit 470171f into TxnLab:main Sep 13, 2024
1 check passed
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