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

Major ModPC Refactor: Mainboards and Componentized Logic #11371

Closed
wants to merge 47 commits into from

Conversation

mystery3525
Copy link
Contributor

@mystery3525 mystery3525 commented Aug 25, 2024

About The Pull Request

Adds the mainboard intermediary object which holds all the PC components like CPU and hard drives. That mainboard is then inside of the modular computer item for Laptops, Tablets, and PDAs, and the modular computer machine for consoles and such.

While I am at it I might as well make it into a component, that's why I need a bunch of new signals to be available so AIs and Cyborgs can participate in the fun.

Mandatory prerequisites:

Todo:

  • Add emag functionality back in
  • Adding NT Messenger back in
  • Figuring out if we want to keep integrity or just let the holder deal with it
  • Testing and making sure Adding/Removing hardware components works
  • Can we add ModPC support on the /mob level? (For IPCs down the line??)
  • Reducing git diffs

Why It's Good For The Game

Moving pretty much all the code away from /obj/item/modular_comptuer/cpu seems like a really good idea and gives us more flexibility to have ModPCs be usable for much more different use-cases (research into /mob ModPCs still in progress)

Turning ModPCs into components allows us to have more flexible code

Testing Photographs and Procedure

Screenshots&Videos

image

Changelog

🆑
add: ModPC mainboards
refactor: Refactors ModPCs to use an intermediary object, mainboard, to store all the logic instead of a subtype
refactor: Adding ModPC logic is now componentized
/:cl:

@github-actions github-actions bot added Administration TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Aug 25, 2024
Copy link

github-actions bot commented Sep 8, 2024

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

1 similar comment
Copy link

github-actions bot commented Sep 8, 2024

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

Copy link

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

Copy link

github-actions bot commented Oct 1, 2024

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

@mystery3525 mystery3525 deleted the mainboard branch November 9, 2024 03:18
@mystery3525 mystery3525 restored the mainboard branch November 9, 2024 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Administration Feature Merge Conflict Refactor TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant