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 unity code stripping for KeyStore #31

Merged

Conversation

nicoeft
Copy link
Contributor

@nicoeft nicoeft commented May 23, 2023

Status Type ⚠️ Core Change Issue
Ready Bug/Hotfix Yes -

Problem

In Game Wallet Login not working in WebGL

Solution

Prevent Unity to strip code from KeyStore. EncryptedKeystoreJson was invalid and that made login fail

Before & After Screenshots

BEFORE:
image

Deploy Notes

Need to build .dll and publish to Solana.Unity-SDK

@GabrielePicco
Copy link
Contributor

LGTM! Thanks @nicoeft

@GabrielePicco GabrielePicco merged commit fc2382c into magicblock-labs:master May 24, 2023
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5056354939

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 72.91%

Totals Coverage Status
Change from base Build 4986780474: 0.0%
Covered Lines: 4730
Relevant Lines: 6189

💛 - Coveralls

xiaolou86 pushed a commit to xiaolou86/Solana.Unity-Core that referenced this pull request Oct 12, 2023
* ✨ Add XNFT Wallet + xNFT WebGLTemplate

- Add XNFTWallet to Runtime codebase and related xNFT js lib to Runtime plugins.
- WebGLTemplates folder with xNFT template to be used when building for xNFT
- Comments  showing how to use it in LoginScreen.cs

* Added WebGLTemplatesExporter + LoginXNFT Button Prefab

* ♻️ Add copy/paste & refactor backpack code

* ✨ Add XNFT Wallet + xNFT WebGLTemplate (magicblock-labs#31)

* ✨ Add XNFT Wallet + xNFT WebGLTemplate

- Add XNFTWallet to Runtime codebase and related xNFT js lib to Runtime plugins.
- WebGLTemplates folder with xNFT template to be used when building for xNFT
- Comments  showing how to use it in LoginScreen.cs

* Added WebGLTemplatesExporter + LoginXNFT Button Prefab

* ➕ Add missing script

* ✨ Update template

* 🙈 Update n.pmignore

* 📄 Update License

* ♻️ Add license meta

* 🎨 Improve link UI

* 🎨 Improve scene

Co-authored-by: Nicolas Francisquelo Tacca <[email protected]>
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