Skip to content

Conversation

@sylviameows
Copy link
Contributor

running the client seems stable and no glaring issues are obvious to me anymore.
however, i did notice that the DestroyItemReset config option wasn't saving and I wasn't able to fix this issue.

[Changes]

  • migrated build phaser to use /ptp [x] [y] [z] [pitch] [yaw] (originally this was supposed to fix a disconnect when using the feature, but the issue was unrelated and has been fixed)

[Fixes]

  • fixed all MiniMessage features to work for 1.21.3
  • fixed Code Palette and Custom Chest GUI rendering for 1.21.3
  • fixed chest highlighting feature 'blinking' in 1.21.3
  • fixed an issue where space and newline tags were affecting the chat bar in negative ways.

@sylviameows sylviameows mentioned this pull request Nov 18, 2024
Copy link
Member

@RedVortexDev RedVortexDev left a comment

Choose a reason for hiding this comment

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

Thank you so much ❤️

@sylviameows
Copy link
Contributor Author

sylviameows commented Nov 19, 2024

resolved the requested changes, here are the changes made:

[Additions]

  • added rate limiting to codeclient sent commands (thanks red!)
  • added /ccclearqueue to clear the command queue when it gets too long.

[Changes]

  • enabled build phaser in build mode.

[Fixes]

  • fixed DestroyItemMode config option not saving.
  • re enabled the /preview command as it was still disabled from initial migration.

Copy link
Member

@RedVortexDev RedVortexDev left a comment

Choose a reason for hiding this comment

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

a bit about command queue

Copy link
Member

@RedVortexDev RedVortexDev left a comment

Choose a reason for hiding this comment

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

LGTM, merging to 1.21 branch.

@RedVortexDev RedVortexDev merged commit b239807 into DFOnline:feat/1.21 Nov 19, 2024
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.

2 participants