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

Port Drop on Slip #54

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Port Drop on Slip #54

wants to merge 7 commits into from

Conversation

Finket
Copy link
Contributor

@Finket Finket commented Feb 25, 2024

Description

A Parkstation list item from #2
I have not made any changes to the code, it might be worth looking at the clothing YAMLs to see if anything should be added or changed.


Todo


Changelog

🆑

  • add: Added chance to drop certain items when slipped

@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files labels Feb 25, 2024
@Finket Finket changed the title Port over drop on slip files Port over drop on slip Feb 25, 2024
@Finket Finket mentioned this pull request Feb 26, 2024
2 tasks
@DEATHB4DEFEAT DEATHB4DEFEAT mentioned this pull request Mar 10, 2024
74 tasks
@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Mar 16, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Mar 18, 2024
@DEATHB4DEFEAT DEATHB4DEFEAT changed the title Port over drop on slip Port Drop on Slip Mar 28, 2024
Copy link
Member

@Pspritechologist Pspritechologist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be using a hardcoded method to check for pockets, etc., and overall needs to be done in a better way. Probably put off merging this until I give a pass of handling it differently.

namespace Content.Server.SimpleStation14.Slippery;

/// <summary>
/// Uses provided chance to try and drop the item when slipped, if equipped.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// Uses provided chance to try and drop the item when slipped, if equipped.
/// Marks this item as one that may be dropped when its wearer slips with it equipped.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Jun 3, 2024
Copy link
Contributor

github-actions bot commented Jun 3, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

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: Localization Changes any ftl files Changes: YML Changes any yml files Status: Merge Conflict FIX YOUR PR AAAGH Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants