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

Don't drop integrated armors on death #60968

Merged
merged 2 commits into from
Sep 14, 2022

Conversation

Venera3
Copy link
Member

@Venera3 Venera3 commented Sep 13, 2022

Summary

Bugfixes "Don't drop integrated armor on death"

Purpose of change

Fixes #57646

Describe the solution

Don't add INTEGRATED items to the dump-worn-items-on-death vector.

Describe alternatives you've considered

None

Testing

Killed Apis before the fix, could loot their skin.
Killed Apis after the fix, could only loot their cold, dead corpse.
Felt bad both times, tho.

Additional context

Remember, kids, complaining about things on Discord works sometimes!

@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. [C++] Changes (can be) made in C++. Previously named `Code` [Markdown] Markdown issues and PRs <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Sep 13, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Sep 13, 2022
@dseguin dseguin merged commit 8ef2dff into CleverRaven:master Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` <Documentation> Design documents, internal info, guides and help. json-styled JSON lint passed, label assigned by github actions [Markdown] Markdown issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

I'm wearing Apis as a skin-suit
2 participants