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

No Uniform, No Helmet, No Service #1040

Merged
merged 3 commits into from
Mar 16, 2023
Merged

No Uniform, No Helmet, No Service #1040

merged 3 commits into from
Mar 16, 2023

Conversation

Fate-JH
Copy link
Contributor

@Fate-JH Fate-JH commented Feb 28, 2023

For the longest time, the player avatar has been able to set cosmetic items at battle rank 24, but player characters haven't been able to depict cosmetics below battle rank 25. This is tied to the third and final uniform upgrade. Mixing uniforms and cosmetics in the wrong pattern causes player characters to be seen without their firearms. For a while now, too, there's been three known uniform upgrades from the normal one and two unknown uniform upgrades that are mirrors of other nearby upgrades. While one of these previously-unknown upgrades continues to remain unknown, the matter of what to do about cosmetic appearance at battle rank 24 has been handled.

Thanks to Scrawny and Untouchable for discovering this issue with the latter's dress.

Please break fewer things.

@Fate-JH Fate-JH added Bug An issue which impedes or disallows the functions of the project. Should Fix labels Feb 28, 2023
@Mazo
Copy link
Contributor

Mazo commented Feb 28, 2023

👍

@ScrawnyRonnie
Copy link
Contributor

Simple test in sanc worked.
image
Weapons and helmet were also visible before changing the cosmetic appearance options.

Would any further scenarios need to be tested? Zoning in vehicles with BR 24 or > passengers/drivers?

@Dethdeath
Copy link

Both
Saw a double selection pop up right after being promoted to BR 24, went away after actually picking something.

A character appeared to be wielding no weapons after zoning, this was fixed when they swapped armor at a terminal.

@Fate-JH Fate-JH linked an issue Mar 10, 2023 that may be closed by this pull request
@Dethdeath
Copy link

As Scrawny mentioned, the rexo client freeze #1045 still occurs on this build, but using the /helmet command prevents it from happening.
Does additional code need to be added to prevent the freeze?
Maybe force use the /helmet command when players first reach BR24?

@Fate-JH
Copy link
Contributor Author

Fate-JH commented Mar 14, 2023

Still have to go back over this PR and integrate the new information.

@Fate-JH
Copy link
Contributor Author

Fate-JH commented Mar 15, 2023

Should be better.

@Fate-JH Fate-JH merged commit 7e899e9 into psforever:master Mar 16, 2023
@Fate-JH Fate-JH deleted the helmet branch March 16, 2023 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug An issue which impedes or disallows the functions of the project. Should Fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rexo client freeze
4 participants