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

Preview 1.82.19 Hotfix Release #698

Merged
merged 40 commits into from
Mar 11, 2025
Merged

Preview 1.82.19 Hotfix Release #698

merged 40 commits into from
Mar 11, 2025

Conversation

bagusnl
Copy link
Member

@bagusnl bagusnl commented Feb 27, 2025

What's changed? - 1.82.19

  • [Fix] Carousel keep running in background, by @neon-nyan
  • [Fix] [ZZZ GSP] Windowed resolution is in wrong order, by @neon-nyan
  • [Fix] Crash when spawning background activity notification, by @neon-nyan
  • [Fix] [SR Repair] Prevent hash file marked as unused, by @neon-nyan
  • [Fix] Notification not bring up window after click, by @neon-nyan
  • [New] [SR GSP] Deprecate Mobile mode feature, by @bagusnl
    • Due to the last game changes causing the feature to not work anymore and fixing it requires memory injection which we do not do (see contribution guidelines)
    • Users can still toggle the settings, but warning is in place
  • [Fix] Notification toast does not bring up the main window, by @neon-nyan
    • Thanks Microsoft, for another undocumented thing you need to do to have a Windows app working!
  • [Loc] Sync localization, thanks localizers <3
    • Massive sync up for ko-KR locale, thanks Youras65, DY3413, @despenser08 (sorry if I missed any one)
  • [Doc] Updated legal documentations to match up the year, by @bagusnl

march7tpn

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

Copy link

github-actions bot commented Feb 27, 2025

Qodana for .NET

13 new problems were found

Inspection name Severity Problems
Possible 'System.NullReferenceException' 🔶 Warning 4
Possible null reference argument for a parameter. 🔶 Warning 2
Unused local variable 🔶 Warning 2
Conditional access qualifier expression is not null according to nullable reference types' annotations 🔶 Warning 1
'??' condition is known to be null or not null 🔶 Warning 1
Non-accessed positional property (non-private accessibility) 🔶 Warning 1
Redundant using directive 🔶 Warning 1
Variable can be declared as non-nullable 🔶 Warning 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

using Microsoft.UI.Xaml;
using System;

Check warning

Code scanning / QDNET

Redundant using directive Warning

Using directive is not required by the code and can be safely removed

// Get the mark file's relative path
string relativePathDir = Path.GetDirectoryName(localFileInfo.RelativePath) ?? "";
string relativePathNameNoExt = Path.GetFileNameWithoutExtension(localFileInfo.RelativePath) ?? "";

Check warning

Code scanning / QDNET

'??' condition is known to be null or not null Warning

'??' left operand is never null
neon-nyan and others added 16 commits March 6, 2025 05:48
This to ensure that the main project, Hi3Helper.Win32 can be used as is to any Windows version, since ToastCOM requires WinRT and use a specific TFM for Windows 10 build 1809
Backport a762be2

Co-authored-by: Kemal Setya Adhi <[email protected]>
Due to the changes in the game mechanism, enabling mobile mode (touchscreen) in HSR requires memory injection in which Collapse does not allow (read Contrib Guidelines). So the feature will be disabled for now.

Closes #697
Also renamed rtf license name since its unrecognized by GitHub
@bagusnl bagusnl marked this pull request as ready for review March 8, 2025 22:21
@bagusnl
Copy link
Member Author

bagusnl commented Mar 8, 2025

Ready for publish @CollapseLauncher/core-maintainers

transifex-integration bot and others added 13 commits March 8, 2025 23:38
100% reviewed source file: 'en_US.json'
on 'es_419'.
100% reviewed source file: 'en_US.json'
on 'zh_CN'.
92% of minimum 75% reviewed source file: 'en_US.json'
on 'uk_UA'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
100% reviewed source file: 'en_US.json'
on 'id_ID'.
88% of minimum 75% reviewed source file: 'en_US.json'
on 'ru_RU'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
99% of minimum 75% reviewed source file: 'en_US.json'
on 'ja_JP'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
99% of minimum 75% reviewed source file: 'en_US.json'
on 'fr_FR'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
96% of minimum 75% reviewed source file: 'en_US.json'
on 'ko_KR'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
86% of minimum 75% reviewed source file: 'en_US.json'
on 'pt_BR'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
99% of minimum 75% reviewed source file: 'en_US.json'
on 'vi_VN'.

Sync of partially translated files: 
untranslated content is included with an empty translation 
or source language content depending on file format
100% reviewed source file: 'en_US.json'
on 'ja_JP'.
+ Also make ToastCOM's callback to work (for real this time).
@bagusnl bagusnl merged commit 66b9c3e into preview Mar 11, 2025
7 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.

3 participants