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

Flowers, Tea, And Clothes #701

Closed

Conversation

SleepyScarecrow
Copy link
Contributor

Description

This adds in a few different things. Most minor is some clothing -- a couple dresses, and some arm warmers to match your favourite pair of coder socks. Flowers are next up-- decorational (for now) plants that I'm going to use for the third thing- tea! I'm going to set up a system for brewing teas, available via a kettle for bartenders, where you shove in the flowers and some water, heat it up, and bam, tea. This tea may be just pure rp, or potentially I'll make it give some effects.
Don't merge this yet, I'm setting it here for drafting sake.


TODO

  • Tea
  • Added Arm Warmers & Dress
  • Added Flowers

Media

None yet, I'll fill this out once it's all said and done.


Changelog

🆑 Tilkku

  • add: Azaleas
  • add: Arm Warmers
  • tweak: Tweaked fun
  • fix: Fixed fun!
  • remove: Removed fun :(

@github-actions github-actions bot added Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files labels Aug 8, 2024
Copy link
Contributor

github-actions bot commented Aug 8, 2024

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

Resources/Textures/Clothing/Uniforms/Costume/azaleadress.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Clothing/Uniforms/Costume/azaleadressblue.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Clothing/Uniforms/Costume/azaleadressred.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Clothing/Uniforms/Costume/azaleadresswhite.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Objects/Consumable/Drinks/electrickettle.rsi

State Old New Status
electrickettle Added
electrickettle1 Added
electrickettle2 Added
electrickettle3 Added
electrickettle4 Added
electrickettle5 Added
electrickettle6 Added
electrickettleboiling Added
inhand-left Added
inhand-right Added

Resources/Textures/Objects/Consumable/Drinks/kettle.rsi

State Old New Status
inhand-left Added
inhand-right Added
kettle Added
kettle1 Added
kettle2 Added
kettle3 Added

Resources/Textures/Objects/Specific/Hydroponics/Flowers/azalea.rsi

State Old New Status
dead Added
equipped-HELMET Added
flower Added
harvest Added
seed Added
stage-1 Added
stage-2 Added
stage-3 Added

Resources/Textures/Objects/Specific/Hydroponics/Flowers/azaleablue.rsi

State Old New Status
dead Added
equipped-HELMET Added
flower Added
harvest Added
seed Added
stage-1 Added
stage-2 Added
stage-3 Added

Resources/Textures/Objects/Specific/Hydroponics/Flowers/azaleared.rsi

State Old New Status
dead Added
equipped-HELMET Added
flower Added
harvest Added
seed Added
stage-1 Added
stage-2 Added
stage-3 Added

Resources/Textures/Objects/Specific/Hydroponics/Flowers/azaleawhite.rsi

State Old New Status
dead Added
equipped-HELMET Added
flower Added
harvest Added
seed Added
stage-1 Added
stage-2 Added
stage-3 Added

Edit: diff updated after a121ddc

@SleepyScarecrow
Copy link
Contributor Author

SleepyScarecrow commented Aug 8, 2024

40 commits
should probably stop reusing the same branch :blunt:

@angelofallars
Copy link
Contributor

40 commits should probably stop reusing the same branch :blunt:

Not a problem. Every PR gets squashed into one commit when merging. I'd personally prefer to see all commits so we can examine changes in depth.

Copy link
Contributor

@DangerRevolution DangerRevolution left a comment

Choose a reason for hiding this comment

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

draft PR but I am sigma.


- type: entity
parent: ClothingHandsArmWarmers
id: ClothingHandsArmWarmersValid
Copy link
Contributor

Choose a reason for hiding this comment

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

no

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no to the item in general or just id:valid

Resources/Prototypes/Entities/Objects/Flowers/flowers.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Hydroponics/seeds.yml Outdated Show resolved Hide resolved
@github-actions github-actions bot added Changes: Audio Changes any ogg files Changes: C# Changes any cs files labels Aug 9, 2024
@SleepyScarecrow
Copy link
Contributor Author

if you care to review this draft still do not pay attention to the c#, only reason I committed it was so a friend could see (i was too lazy to make a patch)

@SleepyScarecrow
Copy link
Contributor Author

had the realisation that oh god dammit I can't acutally base this off microwaves and I have to write it all from scratch :blunt:

Copy link
Contributor

github-actions bot commented Sep 7, 2024

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

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Sep 7, 2024
@github-actions github-actions bot removed Changes: YML Changes any yml files Changes: Localization Changes any ftl files Changes: Audio Changes any ogg files Changes: Sprite Changes any png or json in an RSI Changes: C# Changes any cs files labels Sep 10, 2024
@SleepyScarecrow
Copy link
Contributor Author

gonna redo this shit

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Merge Conflict FIX YOUR PR AAAGH
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants