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

Vendor icon_state fix #3954

Merged
merged 1 commit into from
Jul 22, 2023
Merged

Conversation

ihatethisengine
Copy link
Contributor

@ihatethisengine ihatethisengine commented Jul 20, 2023

About the pull request

I don't know how no one noticed that but some of the vendors were stuck at "_vend" icon_state after first vending. Now it works as intended (hopefully).

Explain why it's good for the game

Bug bad

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 ihatethisengine
fix: vendors icon_state resets after vending as intended.
/:cl:

@github-actions
Copy link
Contributor

You currently have a negative Fix/Feature pull request delta of -1. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score.

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Jul 20, 2023
@ihatethisengine
Copy link
Contributor Author

I tested snowflake vendor and lifeboats vendors, both worked fine, not sure what else should I test.

@ihatethisengine
Copy link
Contributor Author

Wall medvendors also bugged and this should fix it

@harryob harryob added this pull request to the merge queue Jul 22, 2023
Merged via the queue into cmss13-devs:master with commit 5d34ae6 Jul 22, 2023
28 checks passed
@github-actions
Copy link
Contributor

Your Fix/Feature pull request delta is now above zero (2). Feel free to make Feature/Balance PRs.

github-actions bot added a commit that referenced this pull request Jul 22, 2023
cm13-github added a commit that referenced this pull request Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants