Skip to content

Conversation

@Quantumrunner
Copy link
Collaborator

No description provided.

Added UI options and logic to change resolution.
… Unity Native File Picker Plugin instead of StandaloneFileBrowser.
…ction since other plugin is for android ios only.
Android Zip Import, Build Pipeline Improvements & Website Optimization
Added additional error handling for SaveManager
# Conflicts:
#	.gitignore
#	unity/Assets/StreamingAssets/text/Localization.Chinese.txt
#	unity/Assets/StreamingAssets/text/Localization.Czech.txt
#	unity/Assets/StreamingAssets/text/Localization.English.txt
#	unity/Assets/StreamingAssets/text/Localization.French.txt
#	unity/Assets/StreamingAssets/text/Localization.German.txt
#	unity/Assets/StreamingAssets/text/Localization.Italian.txt
#	unity/Assets/StreamingAssets/text/Localization.Japanese.txt
#	unity/Assets/StreamingAssets/text/Localization.Korean.txt
#	unity/Assets/StreamingAssets/text/Localization.Polish.txt
#	unity/Assets/StreamingAssets/text/Localization.Portuguese.txt
#	unity/Assets/StreamingAssets/text/Localization.Russian.txt
#	unity/Assets/StreamingAssets/text/Localization.Spanish.txt
Replaced `Start-Process -Wait` with `$Process.WaitForExit()` in [build.ps1](cci:7://file:///e:/Eigene%20Dokumente/Max/Programmierung/Projekte/valkyrie/build.ps1:0:0-0:0).
The previous method waited for the entire process tree, causing the build to hang for 10 minutes due to lingering `VBCSCompiler.exe` processes spawned by Unity.
The [QuestGetTranslations](cci:1://file:///e:/Eigene%20Dokumente/Max/Programmierung/Projekte/valkyrie/unity/Assets/Scripts/UI/Screens/QuestSelectionScreen.cs:1023:8-1055:9) method had a logic error where the result of `TryGetValue` for the synopsis was negated (`!`).
This caused the code to believe the translation was NOT found when it actually WAS found, ensuring the fallback to the default language always executed.
This commit removes the incorrect negation, allowing local translations to be used when available.
@Quantumrunner Quantumrunner merged commit c5718b9 into master Dec 11, 2025
2 checks 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.

2 participants