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

[MIRROR] Fix OPFOR bluescreen #5203

Merged
merged 2 commits into from
Nov 29, 2024
Merged

[MIRROR] Fix OPFOR bluescreen #5203

merged 2 commits into from
Nov 29, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: NovaSector/NovaSector#4640

i'm really running out of energy to write descriptive commit message

About The Pull Request

Fixes a tgui bluescreen from getting a NaN sent to the backend (due to lack of a step argument) which failed to get sanitized because the original argument was used instead of the sanitized one (and it also wasn't sanitized properly even if it was)
Fixes #4541
Fixes #4271

How This Contributes To The Nova Sector Roleplay Experience

bugn't!

Proof of Testing

Screenshots/Videos
2024-11-27.00-21-04.mp4

i hate how unresponsive (or incredibly oversensitive) these damn sliders are for small values but that's not my problem

Changelog

🆑 FlufflesTheDog
fix: OPFOR equipment quantity can now actually be adjusted
/:cl:

FlufflesTheDog and others added 2 commits November 27, 2024 22:00
fimx

i'm really running out of energy to write descriptive commit message
@FeenieRU FeenieRU merged commit 705a392 into master Nov 29, 2024
32 checks passed
AnywayFarus added a commit that referenced this pull request Nov 29, 2024
@Iajret Iajret deleted the upstream-mirror-4640 branch December 21, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants