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

[7.2] Full Conduit Rework/Revisit #958

Draft
wants to merge 23 commits into
base: dev/1.21.1
Choose a base branch
from

Conversation

Rover656
Copy link
Member

@Rover656 Rover656 commented Jan 3, 2025

Description

This is an entire-system overhaul for conduits, bringing back as many missing 1.12 features as I can and making sure conduits are as polished as they can be.

This PR will not include the visual redesign of the block or the UI, and as such will strive for parity with previous versions of the GUI/block.

This PR is not ready for feedback :)

Breaking Changes

List any breaking changes in this section, such as: changed/removed APIs, changed or removed items/blocks or modifications to recipes and gameplay mechanics.

Major API breaks for mods that add or interact with conduits.
Has supporting logic for loading existing conduit bundles, but it is not saved in a backward-compatible format (i.e. 7.2 cannot be safely downgraded from)

Checklist

  • My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
  • I have made corresponding changes to the documentation.
  • My changes are ready for review from a contributor.

@Rover656 Rover656 added Area-Backend Backend work not usually visible to players. Area-Conduits Everything involving conduits MC-1.21.1 labels Jan 3, 2025
@Rover656 Rover656 force-pushed the chore/conduit-connections branch 2 times, most recently from 7c5d8d8 to f13213b Compare January 8, 2025 11:56
@Rover656 Rover656 changed the title [7.2] Conduit Block Rewrite [7.2] Full Conduit Rework/Revisit Jan 9, 2025
@Rover656 Rover656 force-pushed the chore/conduit-connections branch from 3e7b27a to aba81ea Compare January 11, 2025 02:31
Still need to address many TODOs and GUI
- Tested a possible accessibility improvement
- Copied over clone item stack
- Aiming shapes now only show the conduit you're aiming at, rather than outlining the entire bundle stuff.
- Fixed facades not working (sync and model issues)
- Override particles for bundles due to a crash - need to implement this properly
- Facades can now be placed standalone, without the need for an existing bundle.
- Fixed a crash when breaking an RS node
I'm not interested in documenting all the work and in progress changes here
Also optimised redstone conduit updates a bit.
@Rover656 Rover656 force-pushed the chore/conduit-connections branch from 73a36c3 to 32925d2 Compare January 11, 2025 23:31
@Rover656 Rover656 added this to the 7.2 Update milestone Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Backend Backend work not usually visible to players. Area-Conduits Everything involving conduits MC-1.21.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant