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

Fix Crime Assist incorrect entires and add a test #624

Merged
merged 3 commits into from
Dec 31, 2023

Conversation

DebugOk
Copy link
Contributor

@DebugOk DebugOk commented Dec 31, 2023

About the PR

This adds a test that ensures all CrimeAssistPage's are valid, and will prevent any future issues where the program can display locstrings or get stuck on a page and error out.

@DebugOk DebugOk requested a review from Colin-Tel as a code owner December 31, 2023 16:04
@github-actions github-actions bot added S: Needs Review Changes: C# Changes any cs files Changes: YML Changes any yml files labels Dec 31, 2023
@DebugOk DebugOk merged commit e09112b into DeltaV-Station:master Dec 31, 2023
10 checks passed
DebugOk added a commit to DebugOk/Delta-v that referenced this pull request Jan 20, 2024
* Create CrimeassistTest.cs

* Fix invalid crimeasssist entries

* Clean up test code

(cherry picked from commit e09112b)
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
# Description

Done by request from several people who have some issues with the sound
mixing of certain items being very off. I did a little digging, and
discovered that the heart of the issue is that id: BaseItem had for
whatever utterly strange reason, set the volume for every item not
overriding it to be above people's sound settings, resulting in pickup
and drop sounds being strangely inconsistent with all other volumes. It
got so bad that some people reportedly couldn't play the game.

# Changelog

:cl:
- fix: Fixed Tools, Drink Containers, Lockers, and all base items having
inconsistent sound settings.
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant