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

[Upstream] HoP Expansion Pack - Papers Please Edition #2434

Merged
merged 7 commits into from
Jul 7, 2023

Conversation

Bobbanz1
Copy link
Contributor

@Bobbanz1 Bobbanz1 commented May 29, 2023

About The Pull Request

Pulls from Upstream the Gigantic Paperwork Rework

Also included is this:

This PR also merges the fax machines added to most of our maps from:

Alongside merging the changes made in this PR to the photocopier and integrating the altered code with the Beestation photocopier machine:

IT also ports from TG-Station the fix that makes the forms created from the photocopiers actually work.

(To whoever decides to go through and review all this stuff, I'm sorry)

Why It's Good For The Game

A more up to date version of the Paperwork Module, allows people to see the result of what they're entering on a paper, fax machines to annoy admins with and other fun stuff!

Testing Photographs and Procedure

Screenshots&Videos

204471147-20920092-bdf3-403e-8f5d-b2f684be6f25

Changelog

🆑Bobbanz1, Timberpoes, SteelSlayer, twilightwanderer, ShizCalev, timothymtorres, Imaginos16, dragomagol, Jacquerel, Mothblocks, ThePotato97, LemonInTheDark, itsmeow
del: Removed the ability to scan paper directly into the modular computer notes app as part of paper code rewrite.
qol: AIs now see the full paper interface when players show them papers through the camera, exactly what's written on the paper, fully rendered.
refactor: Paper code has been completely refactored.
fix: Fixes signatures not working at all. Signatures now only work when %s or %sign is put into an [___] input field.
fix: Input fields are now fixed and also work again.
fix: Fixes broken command report formatting for station goals.
fix: Fixes markdown paragraphing not working in paper code.
refactor: Photocopiers now use TGUI
tweak: You can insert any type of toner cartridge into photocopiers, and regardless of the currently inserted toner's charge level.
tweak: Toner cartridges can be ejected from photocopiers at any time
add: Added a system of forms and the ability to print them
fix: Hints in bureaucracy forms
add: Added new VOID stamp
fix: Photocopier paperforms
add: Fax Machine
fix: Fixes faxes playing the "fax_contain" animation and makes it actually use the "fax_receive" animation when receiving.
imageadd: Resprites fax machines!
admin: fax messages and book printings are now logged in paper.log
admin: renaming fax machines is logged in game log
add: You can now send photos or trading cards through the fax machine.
add: You can now hack the wires of a fax machine to have it throw items harder or accept additional flat input items such as material sheets, ID cards, and pizza slices.
add: Ability to send and receive administrator fax messages.
admin: Panel for easy viewing and sending messages to faxes.
fix: Fixes a crash of TGUI fax panel when there is no fax in the network
fix: Fixes a crash of the Fax Manager panel when there is no fax machines in the network.
refactor: Rotating stamps on paper by holding shift and moving your mouse.
fix: You can no longer remove the toner cartridge from a printer while it's busy printing.
fix: Fixes broken input fields on photocopied paper.
fix: Fixed exploit in CentCom faxes
/:cl:

Copy link
Contributor

@Bokkiewokkie Bokkiewokkie left a comment

Choose a reason for hiding this comment

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

Mapping review:
You should swap the tables in Tycoon engineering there, they can't actually reach the fax machine in that corner.
Also, please give the Atlas bridge a pen for that paper bin and a command folder on a table somewhere to sort their documents 🙏

@Bokkiewokkie Bokkiewokkie added the Mass Map Change This PR will likely change two or more maps in order to implement its primary feature. label Jun 2, 2023
@Bobbanz1
Copy link
Contributor Author

Bobbanz1 commented Jun 2, 2023

Mapping review:
You should swap the tables in Tycoon engineering there, they can't actually reach the fax machine in that corner.

That’s there by tradition but I’ll make it happen.

Also, please give the Atlas bridge a pen for that paper bin and a command folder on a table somewhere to sort their documents 🙏

I’ll make it happen later today.

@Bokkiewokkie Bokkiewokkie added Feature Active Test Merge Update PRs that pull updates from upstream labels Jun 2, 2023
Bokkiewokkie
Bokkiewokkie previously approved these changes Jun 9, 2023
Copy link
Contributor

@Bokkiewokkie Bokkiewokkie left a comment

Choose a reason for hiding this comment

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

Mapping approved

Copy link
Member

@covertcorvid covertcorvid left a comment

Choose a reason for hiding this comment

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

LGTM

@Bokkiewokkie Bokkiewokkie merged commit 0b5abc1 into BeeStation:master Jul 7, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Active Test Merge Administration Feature Mapping Mass Map Change This PR will likely change two or more maps in order to implement its primary feature. Sprites TGUI-Changes Update PRs that pull updates from upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants