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

Stinky trait #594

Merged
merged 16 commits into from
Dec 1, 2023
Merged

Stinky trait #594

merged 16 commits into from
Dec 1, 2023

Conversation

dvir001
Copy link
Contributor

@dvir001 dvir001 commented Nov 17, 2023

About the PR

Added a new trait, stinky.

Gives you an examine action that tells you and others that you stinks.
Adds a popup every 5-10 minutes to remind people you stink if there around you.
Allow you to keep an air freshener in your pocket to remove the effect while you have it.

Thats all.

Why / Balance

Some lizards will want this.

Technical details

New trait, mostly yml and C#
There is a disabled option to use any type of gas with the trait, cannot be used unless you add gas type and activate the bool.

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame,

Breaking changes

N/A

Changelog
🆑 dvir01

  • add: Some of our crew started skipping on showers to save water.
  • add: Added the air freshener to the vendomat vending machine to allow our water saving crew to not smell so bad.

Copy link
Contributor

github-actions bot commented Nov 17, 2023

RSI Diff Bot; head commit 580fbf4 merging into de46891
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_NF/Objects/Misc/airfreshener.rsi

State Old New Status
icon Added

Edit: diff updated after 580fbf4

@Cheackraze
Copy link
Member

I dont think having an innate ability to kill people is a good idea

@dvir001
Copy link
Contributor Author

dvir001 commented Nov 18, 2023

I dont think having an innate ability to kill people is a good idea

It takes more then 2 hours in a small room to be noticeable, much more longer then any dead body takes.
Either way I lowered the amount even more.

    - type: StinkyTrait
      timeBetweenIncidents: 5, 15 # Incident every X-Y
      durationOfIncident: 0.1, 0.1 # 0.01 per Incident

We can test it more in game and if you still think someone using it actually killing any one you can change the timeBetweenIncidents to be 300, 600 so its 0 real effect other then adding miasma particles.

@dvir001
Copy link
Contributor Author

dvir001 commented Nov 19, 2023

Miasma is set to false, popup added.

@dvir001
Copy link
Contributor Author

dvir001 commented Nov 22, 2023

Reworked it for the last time, Its working as it need to be working, no gas per base trait, just the harmless popup.

Copy link
Member

@Cheackraze Cheackraze left a comment

Choose a reason for hiding this comment

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

This still has all the code in it for atmos, please remove

@dvir001
Copy link
Contributor Author

dvir001 commented Nov 25, 2023

This still has all the code in it for atmos, please remove

The code block?
Ill like to use it for a mob later on that will show up on events, its fully disabled for the stinky trait, and even if was set on without having to say what gas you want it to use its still auto off.

If its still an issue I can un comment it till much later, but ill like to have it ready for later please.

@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Nov 25, 2023
@Cheackraze
Copy link
Member

This still has all the code in it for atmos, please remove

The code block? Ill like to use it for a mob later on that will show up on events, its fully disabled for the stinky trait, and even if was set on without having to say what gas you want it to use its still auto off.

If its still an issue I can un comment it till much later, but ill like to have it ready for later please.

i think if you want this functionality for a future system, it should be implemented in that system instead and leave the pure trait code to be just the popup + air freshener

@dvir001
Copy link
Contributor Author

dvir001 commented Nov 26, 2023

This still has all the code in it for atmos, please remove

The code block? Ill like to use it for a mob later on that will show up on events, its fully disabled for the stinky trait, and even if was set on without having to say what gas you want it to use its still auto off.
If its still an issue I can un comment it till much later, but ill like to have it ready for later please.

i think if you want this functionality for a future system, it should be implemented in that system instead and leave the pure trait code to be just the popup + air freshener

Sigh, fixed.

@Cheackraze Cheackraze merged commit d60b10d into new-frontiers-14:master Dec 1, 2023
10 of 11 checks passed
FrontierATC added a commit that referenced this pull request Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants