Skip to content

IOS build error fix#718

Merged
robGG1997 merged 44 commits into
mainfrom
rob/ios-build-error-fix-699
Nov 16, 2023
Merged

IOS build error fix#718
robGG1997 merged 44 commits into
mainfrom
rob/ios-build-error-fix-699

Conversation

@robGG1997
Copy link
Copy Markdown
Contributor

@robGG1997 robGG1997 commented Nov 9, 2023

Closes #699

Make sure Strip Engine Code is set to false and Code Stripping is Set to minimal in Player Settings

@robGG1997 robGG1997 added this to the 2.5.1 Release milestone Nov 9, 2023
@robGG1997 robGG1997 self-assigned this Nov 9, 2023
@robGG1997 robGG1997 force-pushed the rob/ios-build-error-fix-699 branch from 79dfcfc to 08fd513 Compare November 9, 2023 09:30
@robGG1997 robGG1997 added the ready-to-merge Ready to Merge PR - this'll trigger required checks label Nov 14, 2023
Copy link
Copy Markdown
Contributor

@juans-chainsafe juans-chainsafe left a comment

Choose a reason for hiding this comment

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

Issue is fixed! But is not working the “Redirect to Wallet” it redirects to the wallet but it doesnt prompt to connect to the app

Comment thread Packages/io.chainsafe.web3-unity/Tests/Runtime/EvmTests.cs
@sneakzttv sneakzttv linked an issue Nov 15, 2023 that may be closed by this pull request
17 tasks
@robGG1997 robGG1997 added ready-to-merge Ready to Merge PR - this'll trigger required checks and removed ready-to-merge Ready to Merge PR - this'll trigger required checks labels Nov 16, 2023
@robGG1997 robGG1997 closed this Nov 16, 2023
@robGG1997 robGG1997 reopened this Nov 16, 2023
@robGG1997 robGG1997 added ready-to-merge Ready to Merge PR - this'll trigger required checks and removed ready-to-merge Ready to Merge PR - this'll trigger required checks labels Nov 16, 2023
@robGG1997 robGG1997 merged commit a81e54a into main Nov 16, 2023
@robGG1997 robGG1997 deleted the rob/ios-build-error-fix-699 branch November 16, 2023 11:42
rob1997 pushed a commit that referenced this pull request Jan 16, 2025
* Revert "automated duplicating imported samples modifications (#595)"

This reverts commit a555894.

* setup for IOS build based WC's feedback

* added custom websocket implementation

* refactor and cleanup

* moved web socket as plugin in packages

* fixed ws destroy error

* reuse connection builder after a logout

* ignore generated data files in sample project

* removed some unwanted changes and ignored files

* removed editor folder from assets and added docs for connection builder in WalletConnectConfig.cs

* Auto-duplicate Packages Samples

* Published Solution Dependencies to Package Libraries as DLLs

* transfer erc 1155 test now passing

* Published Solution Dependencies to Package Libraries as DLLs

* Published Solution Dependencies to Package Libraries as DLLs

---------

Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>
sergeypanin1994 pushed a commit to sergeypanin1994/web3.unity that referenced this pull request Mar 16, 2025
* Revert "automated duplicating imported samples modifications (ChainSafe#595)"

This reverts commit a555894.

* setup for IOS build based WC's feedback

* added custom websocket implementation

* refactor and cleanup

* moved web socket as plugin in packages

* fixed ws destroy error

* reuse connection builder after a logout

* ignore generated data files in sample project

* removed some unwanted changes and ignored files

* removed editor folder from assets and added docs for connection builder in WalletConnectConfig.cs

* Auto-duplicate Packages Samples

* Published Solution Dependencies to Package Libraries as DLLs

* transfer erc 1155 test now passing

* Published Solution Dependencies to Package Libraries as DLLs

* Published Solution Dependencies to Package Libraries as DLLs

---------

Co-authored-by: robGG1997 <robGG1997@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Ready to Merge PR - this'll trigger required checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remote wallet not working on iOS build Issue Tracker 2.5.1 Release

4 participants