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

Implementation of Module singularities #1685

Closed
wants to merge 16 commits into from

Conversation

KorewaLidesu
Copy link
Contributor

@KorewaLidesu KorewaLidesu commented Mar 10, 2023

Testing procedure has done. All test successed.
Need players to test if there is other issue.

PR: Implementing Modules to replace readFromNBT & writeToNBT

Resolve #1679 (comment)

Thing has done:

  • PipeBlockRequestTable
  • PipeFluidExtractor
  • PipeFluidInsertion
  • PipeFluidSatellite
  • PipeFluidSupplierMk2
  • PipeItemsFirewall
  • PipeItemsFluidSupplier
  • PipeItemsInvSysConnector
  • PipeItemsSatelliteLogistics
  • PipeItemsSystemDestinationLogistics
  • PipeItemsSystemEntranceLogistics
  • PipeLogisticsChassis
  • PipeItemsRequestLogisticsMk2

…ow it to be stored properly on early loading stage.
…, PipeFluidSatellite, PipeFluidSupplierMk2, PipeItemsFluidSupplier. Change conditions stuff in PipeItemsSatelliteLogistics (Commit for: RS485#1679)
…temDestinationLogistics, PipeItemsSystemEntranceLogistics, some small enhancement
@KorewaLidesu KorewaLidesu changed the title Implementation of Module singularities Implementation of Module singularities and small code typing touch Mar 11, 2023
This reverts commit 4d3c4e3.
…LogisticsMk2, add other features that I cant type all out
@KorewaLidesu KorewaLidesu changed the title Implementation of Module singularities and small code typing touch Implementation of Module singularities Mar 18, 2023
@KorewaLidesu
Copy link
Contributor Author

The PR is ready to review!

# Conflicts:
#	common/logisticspipes/gui/GuiFirewall.java
#	common/logisticspipes/gui/orderer/GuiRequestTable.java
#	common/logisticspipes/pipes/PipeItemsFirewall.java
#	common/logisticspipes/pipes/PipeItemsInvSysConnector.java
#	common/logisticspipes/pipes/PipeItemsSystemDestinationLogistics.java
#	common/logisticspipes/pipes/PipeLogisticsChassis.java
@KorewaLidesu
Copy link
Contributor Author

Merged with upstream!

Copy link
Member

@bziemons bziemons left a comment

Choose a reason for hiding this comment

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

Sorry for the silence about this PR. As I said it is a bit big for a first contribution and for my time to review it thoroughly in one go.

@KorewaLidesu
Copy link
Contributor Author

KorewaLidesu commented Apr 28, 2023

Should be good to go then
Will check for in-game later (should not affect too much) Test done, look good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Pipe's NBT don't save properly
2 participants