Skip to content

Commit 45ac0cb

Browse files
committed
[Docs] Improve Unity Build Instructions (#6287)
Closes TOOL-3456 <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating descriptions and improving the clarity of build instructions for Unity and troubleshooting OAuth redirects. ### Detailed summary - Fixed a typo in the `description` for the Unreal Engine link. - Changed section title from `General` to `All Target Platforms`. - Clarified build settings for IL2CPP Code Generation. - Added details about OAuth redirect troubleshooting, including best practices for bundle IDs. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
1 parent 8610b46 commit 45ac0cb

File tree

2 files changed

+14
-4
lines changed
  • apps/portal/src/app

2 files changed

+14
-4
lines changed

apps/portal/src/app/connect/blockchain-api/page.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ Perfomant, reliable and type safe API to read write to any contract on any EVM c
4545
title="Unreal Engine"
4646
icon={UnrealEngineIcon}
4747
href="/unreal-engine"
48-
description="Unral blueprints and C++ API"
48+
description="Unreal blueprints and C++ API"
4949
/>
5050

5151
</Stack>

apps/portal/src/app/unity/v5/build-instructions/page.mdx

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ export const metadata = createMetadata({
88

99
# Build Instructions
1010

11-
## General
11+
## All Target Platforms
1212

13-
- **Build Settings:** Use `Smaller (faster) Builds` / `Shorter Build Time`.
1413
- **Player Settings:** Use IL2CPP over Mono when available.
14+
- **Build Settings:** Use `Smaller (faster) Builds` / `Shorter Build Time` for IL2CPP Code Generation.
1515
- **Stripping Level:** Set `Managed Stripping Level` to `Minimal` (`Player Settings` > `Other Settings` > `Optimization`). (Generally not a hard requirement unless using WalletConnect as a wallet provider option.)
1616
- **Strip Engine Code:** Make sure this is turned off.
1717

@@ -47,9 +47,19 @@ No action needed for hosted builds.
4747
## Mobile
4848

4949
- **EDM4U:** Comes with the package, resolves dependencies at runtime. Use `Force Resolve` from `Assets` > `External Dependency Manager` > `Android Resolver`.
50-
- **Redirect Schemes:** Set custom schemes matching your bundle ID in `Plugins/AndroidManifest.xml` or equivalent to ensure OAuth redirects.
50+
- **Redirect Schemes:** Set custom schemes matching your bundle ID in `Plugins/Android/AndroidManifest.xml` or equivalent to ensure OAuth redirects if using social login.
5151
```xml
5252
<!-- Set your bundle id here -->
5353
<!-- Replace 'com.thirdweb.unitysdk' with your desired scheme -->
5454
<data android:scheme="com.thirdweb.unitysdk" />
5555
```
56+
57+
## Troubleshooting OAuth Redirects
58+
59+
If you notice issues with redirects on mobile, for instance hanging after selecting an account when logging in with Google, you did not setup OAuth properly.
60+
Best practice is to use a lowercase bundle id that is the same as your Unity project's application identifier, it must be present in the following places:
61+
- Your `ThirdwebManager` prefab `Bundle ID` field.
62+
- Your `Plugins/AndroidManifest.xml` scheme field as mentioned above (it doesn't have to be our default `AndroidManifest.xml`, it can be in your custom one, just add the scheme).
63+
- Your thirdweb dashboard project settings as one of:
64+
- `Bundle ID Allowlist` field (those automatically added to allowed redirect URIs).
65+
- In-App Wallet Configuration `Allowed redirect URIs` field.

0 commit comments

Comments
 (0)