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

Ab warning dialog #380

Merged
merged 7 commits into from
Jan 30, 2024
Merged

Ab warning dialog #380

merged 7 commits into from
Jan 30, 2024

Conversation

Cryotechnic
Copy link
Collaborator

This PR fixes an issue where the warning dialog that would show if a specific registry key was detected, would show every time the region was loaded. This is now limited to only when the user opens the Honkai: Star Rail settings page, and will appear every time it is loaded.

@Cryotechnic Cryotechnic added the Area: Game Settings Issue labeled for Game Settings related issues label Jan 26, 2024
@Cryotechnic Cryotechnic added this to the 1.73.x milestone Jan 26, 2024
@Cryotechnic Cryotechnic requested a review from bagusnl January 26, 2024 15:40
@Cryotechnic Cryotechnic self-assigned this Jan 26, 2024
Copy link

github-actions bot commented Jan 26, 2024

Qodana for .NET

1 new problem were found

Inspection name Severity Problems
Possible 'System.NullReferenceException' 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

Copy link
Member

@bagusnl bagusnl left a comment

Choose a reason for hiding this comment

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

Overall code looks good, just needs a little attention in the subtitles spacing and nullable context

Hi3Helper.Core/Lang/en_US.json Outdated Show resolved Hide resolved
@bagusnl bagusnl self-requested a review January 30, 2024 04:54
Copy link
Member

@bagusnl bagusnl left a comment

Choose a reason for hiding this comment

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

All good, approb

@Cryotechnic
Copy link
Collaborator Author

All good, approb

image

@Cryotechnic Cryotechnic merged commit ab6b586 into main Jan 30, 2024
4 checks passed
@Cryotechnic Cryotechnic deleted the ab-warning-dialog branch January 30, 2024 04:57
Cryotechnic added a commit that referenced this pull request Feb 1, 2024
* move a/b code to hsr gsp & create new dialog type

* Use ``MainWindow``'s ``XamlRoot`` for dialogs

* Clean-up unused namespaces

* updated locale

* code cleanup

* added nullable to registry a/b detection & typo fixes

---------

Co-authored-by: Kemal Setya Adhi <[email protected]>
neon-nyan added a commit that referenced this pull request Feb 5, 2024
* move a/b code to hsr gsp & create new dialog type

* Use ``MainWindow``'s ``XamlRoot`` for dialogs

* Clean-up unused namespaces

* updated locale

* code cleanup

* added nullable to registry a/b detection & typo fixes

---------

Co-authored-by: Kemal Setya Adhi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Game Settings Issue labeled for Game Settings related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants