meteor-client icon indicating copy to clipboard operation
meteor-client copied to clipboard

Many improvements to AutoSign

Open RealMuffinTime opened this issue 1 year ago • 9 comments

Type of change

  • [x] New feature

Description

  • Remove hardcoded time placeholder from MeteorStarscript.java
  • Added persistent settings to save sign lines to
  • Added Starscript support
  • Added possibility to also apply back side of sign
  • Added possibility to overwrite or keep existent text on sign When not applying anything, open edit screen normally

How Has This Been Tested?

  • Placing many signs, in diffrent situations
  • On a local vanilla server.
  • Some SMP servers.

Checklist:

  • [x] My code follows the style guidelines of this project.
  • [x] I have added comments to my code in more complex areas.
  • [x] I have tested the code in both development and production environments.

RealMuffinTime avatar May 21 '24 22:05 RealMuffinTime

what was wrong with the time placeholder? it looks correct from looking at the code

crosby-moe avatar May 21 '24 22:05 crosby-moe

It was not wrong, but it did always overwrite the time placeholder from the Starscript StandardLib.timeFormat object.

RealMuffinTime avatar May 21 '24 22:05 RealMuffinTime

It was not wrong, but it did always overwrite the time placeholder from the Starscript StandardLib.timeFormat object.

ahh i see

crosby-moe avatar May 21 '24 22:05 crosby-moe

While testing I found some bigger issues, need to look into how to solve these.

RealMuffinTime avatar May 22 '24 17:05 RealMuffinTime

While testing I found some bigger issues, need to look into how to solve these.

what are the issues?

crosby-moe avatar May 22 '24 20:05 crosby-moe

First something unrelated: sign.getEditor() is always null for me?

So the module works fine on a server which accepts sign editing (vanilla). But I also tested it on servers where editing is not possible, this turned out to have negative side effects, while having back sign writing enabled. Double placed signs on 2b2t (2b2t doesn't allow editing), every second sign empty on constantiam (older mc version).

RealMuffinTime avatar May 22 '24 21:05 RealMuffinTime

I tried to use sign.getEditor() to verify if the BlockInteraction was successfull and if the sign could be edited.

RealMuffinTime avatar May 22 '24 21:05 RealMuffinTime

Is checked and tested. Added a warning, if server doesn't support back sign editing.

RealMuffinTime avatar May 31 '24 17:05 RealMuffinTime

Hi, I would really like, to see this merged. :) Also if https://github.com/MeteorDevelopment/starscript/pull/21 is not yet being merged.

Or is there something else holding up this PR?

RealMuffinTime avatar Aug 25 '24 09:08 RealMuffinTime