Skip to content

The BlockDiode now correctly activates the BlockNote#191

Open
Dereku wants to merge 1 commit into
retromcorg:masterfrom
Dereku:issue-dscrd1500880647210795059
Open

The BlockDiode now correctly activates the BlockNote#191
Dereku wants to merge 1 commit into
retromcorg:masterfrom
Dereku:issue-dscrd1500880647210795059

Conversation

@Dereku

@Dereku Dereku commented May 13, 2026

Copy link
Copy Markdown
Contributor

📌 Pull Request

Description

That PR resolves an issue with BlockDiode (Repeater block) not behaving correctly when interacting with BlockNote (Note block)

Related Issues / Discussions

https://discord.com/channels/654195300478746625/688899501162758170/1500880647210795059

Motivation & Context

Bug fix

Type of Change

Please tick all that apply:

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ⚡ Performance improvement
  • 🧩 New feature (non-breaking)
  • 🔧 Refactor / cleanup (no functional changes)
  • 🔥 Crash / exploit fix
  • 📚 Documentation update
  • ❗ Breaking change (may affect plugins or server behavior)

Testing Performed

  • Compiled successfully with mvn clean package
  • Tested on a Beta 1.7.3 server
  • Existing functionality verified
  • Edge cases considered

Test details:
https://discord.com/channels/654195300478746625/688899501162758170/1504122185831420005
0:00 - 0:06 - behavior w/o the patch , 0:06 - 0:11 - behavior with the patch

Also, several tests with the directions of activation did not reveal any negative changes.

Logs / Screenshots (if applicable)

N/A

Compatibility Considerations

  • No known compatibility impact
  • Potential plugin impact (described below)
  • Network / protocol (Netcode) behavior changed
  • Configuration change required

Compatibility Notes

This change may cause an increased number of the BlockRedstoneEvents.

Checklist

Please confirm the following:

  • No unnecessary formatting or whitespace-only changes
  • Changes are compatible with Minecraft Beta 1.7.3
  • No dependencies have been added without discussion with the RetroMC team

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.

1 participant