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

Fix data stick not copying distinct bus from me input bus #2448

Merged

Conversation

omergunr100
Copy link
Contributor

What

Adds the distinct bus field to the copied and pasted nbt.

Suggestion

I suggest unifying the configuration copy pasting to a single interface that lets you:

  1. Copy machine configuration as nbt.
  2. Paste machine configuration as nbt.
  3. Get what tools/items are allowed to copy/paste this machine configuration (if we want to keep this behaviour on the data stick)

(personal opinion warning)
I think the data stick interaction is a bit weird, we have a way of doing exactly that with item behaviours. (also see #2405 )
also copying on left click feels a bit clunky to me, I feel like copying on shift + <copy key> feels better but that's just me.

Also

Fixes #2344

@omergunr100 omergunr100 requested a review from a team as a code owner November 28, 2024 14:10
@screret screret added the type: feature New feature or request label Nov 29, 2024
@omergunr100 omergunr100 requested a review from screret November 29, 2024 23:30
@YoungOnionMC YoungOnionMC merged commit c611a34 into GregTechCEu:1.20.1 Dec 1, 2024
3 checks passed
@omergunr100 omergunr100 deleted the 1.20.1-data-stick-copy-fix branch December 1, 2024 10:48
screret pushed a commit that referenced this pull request Dec 5, 2024
Co-authored-by: YoungOnion <39562198+YoungOnionMC@users.noreply.github.com>
@Spicierspace153 Spicierspace153 added the type: bugfix general bug fixes label Dec 16, 2024
screret pushed a commit that referenced this pull request Dec 28, 2024
Co-authored-by: YoungOnion <39562198+YoungOnionMC@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bugfix general bug fixes type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Data Stick does not copy all settings of ME (Stocking) Input Bus
4 participants