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

[MIRROR] Debug chem synthesizer works & related code cleanup #713

Merged
merged 1 commit into from
Nov 19, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25109
Original PR: tgstation/tgstation#79616

About The Pull Request

  1. Fixes #79610

    It works and the UI has also been updated to match that of chem dispenser and it displays an input list for adding reagents now

    Screenshot (355)

  2. Created a common component for chem dispenser, portable chem dispenser & debug chem synthesizer to share in the UI code.

  3. Moved portable chem mixer to modules/reagents/machinery folder to group them under the chemistry category

Changelog

🆑 SyncIt21
fix: debug chem synthesizer works again. cleaned up chem dispenser, portable chem dispenser & debug chem synthesizer ui code
qol: ui for displaying beaker reagents for debug chem synthesizer has been improved. Now displays input list for adding reagents
/:cl:

… [MDB IGNORE] (#25109)

* [NO GBP] Debug chem synthesizer works & related code cleanup (#79616)

## About The Pull Request
1. Fixes #79610

It works and the UI has also been updated to match that of chem
dispenser and it displays an input list for adding reagents now

![Screenshot
(355)](https://github.com/tgstation/tgstation/assets/110812394/cb6f5a39-0689-44ce-98b4-cd3027bf7188)

2. Created a common component for chem dispenser, portable chem
dispenser & debug chem synthesizer to share in the UI code.

3. Moved portable chem mixer to modules/reagents/machinery folder to
group them under the chemistry category

## Changelog
:cl:
fix: debug chem synthesizer works again. cleaned up chem dispenser,
portable chem dispenser & debug chem synthesizer ui code
qol: ui for displaying beaker reagents for debug chem synthesizer has
been improved. Now displays input list for adding reagents
/:cl:

* [NO GBP] Debug chem synthesizer works & related code cleanup

---------

Co-authored-by: SyncIt21 <[email protected]>
@Iajret Iajret merged commit 9605798 into master Nov 19, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25109 branch November 19, 2023 17:26
AnywayFarus added a commit that referenced this pull request Nov 19, 2023
Iajret pushed a commit that referenced this pull request Jan 31, 2024
* Fixes Cult voting for a leader (#81169)

## About The Pull Request

With the recent changes to polling, it seems like it didn't properly
poll cultists, I fixed that and fixed other issues with it (such as it
telling players that they are signing up to BE a cult leader, when they
arent).

Also fixed the button to put yourself forward as cult leader, it
properly removes and gives itself when necessary.

## Why It's Good For The Game

Cult button works again how cool is that

Closes tgstation/tgstation#80620

## Changelog

:cl:
fix: Cultists can now vote for a Cult leader again.
/:cl:

* Fixes Cult voting for a leader

---------

Co-authored-by: John Willard <[email protected]>
ReezeBL pushed a commit that referenced this pull request Feb 6, 2024
* Fixes Cult voting for a leader (#81169)

## About The Pull Request

With the recent changes to polling, it seems like it didn't properly
poll cultists, I fixed that and fixed other issues with it (such as it
telling players that they are signing up to BE a cult leader, when they
arent).

Also fixed the button to put yourself forward as cult leader, it
properly removes and gives itself when necessary.

## Why It's Good For The Game

Cult button works again how cool is that

Closes tgstation/tgstation#80620

## Changelog

:cl:
fix: Cultists can now vote for a Cult leader again.
/:cl:

* Fixes Cult voting for a leader

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: John Willard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants