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

More Configurability #31

Merged
merged 6 commits into from
Jun 23, 2023
Merged

More Configurability #31

merged 6 commits into from
Jun 23, 2023

Conversation

machiecodes
Copy link
Contributor

Adds two config options:

sendDawnMessage - specifies whether to send the dawn message, makes sense since a "sendSleepingMessage" option is also provided and makes disabling the message more obvious

showTimeUntilDawn - specifies whether the time until dawn should be appended to the sleeping message, made true by default. Allows for greater vanilla parity

Also changed the construction of the sleeping message to use String.format() which would maybe have a tiiiiiiny improvement for memory use lol

Makes disabling the dawn message clearer and makes sense since a "sendSleepingMessage" option is also provided
Specifies whether the time until dawn is appended to the sleep message, also uses String.format() to template the string, maybe better for memory ??
@Steveplays28 Steveplays28 added the enhancement New feature or request / improvement of existing feature label Jun 15, 2023
@Steveplays28
Copy link
Member

Steveplays28 commented Jun 15, 2023

Hey Ricky, thanks for your PR, it looks really good!
I'll take a look at merging and releasing in a bit!

@machiecodes
Copy link
Contributor Author

For sure! Was just perusing mods for a server I'm making and thought of a few features ^^

I could work on making a TPS tracker of some sort like I saw in #7 but I can't guarantee when or if that would get done, probably after I get the server up and running

@Steveplays28
Copy link
Member

That would be neat! Maybe we could depend on Carpet Core, because I believe it has TPS tracker functionality in its API.

@Steveplays28
Copy link
Member

Could you rebase against main? Did some cleanup that I forgot to cherry pick from the 1.20 branch.

@machiecodes
Copy link
Contributor Author

Ope- Completely slipped my mind, let me take care of that ye

Makes disabling the dawn message clearer and makes sense since a "sendSleepingMessage" option is also provided
Specifies whether the time until dawn is appended to the sleep message, also uses String.format() to template the string, maybe better for memory ??
@machiecodes
Copy link
Contributor Author

I think this should do it? I have no idea how rebasing works but I selected my stuff, if this is not right I will just refork it 💀

@Steveplays28
Copy link
Member

That seems to be fine! I'll merge and release it sometime soon.
Still need to look into your issue.

@Steveplays28 Steveplays28 merged commit 3420dc9 into steves-underwater-paradise:main Jun 23, 2023
@Steveplays28
Copy link
Member

Thanks! Will look into your issue sometime next week.

@Steveplays28 Steveplays28 added this to the 1.7.2 milestone Jun 23, 2023
Steveplays28 pushed a commit that referenced this pull request Jul 28, 2023
Co-authored-by: Steveplays28 <djspaargaren@outlook.com>
(cherry picked from commit 3420dc9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request / improvement of existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants