[Feature-Request] Variable for "content" in webhooks about docker-palworld-dedicated-server

[Jadiction]

@Jadiction Please go ahead and test it and give me feedback.

Works good! Thank you!

To me, this looks better

rather than repeating the "Status update" when the information in the embed shows everything needed

[Jadiction]

WEBHOOK_ALL_TITLE will never be not set, because it goes in the Dockerfile for default-sets and Backwards-Compability.
This makes things more complicated. But when i understand you correctly you would offer:

Ah true, after thinking about it, the first solution wouldn't work, my bad. Was thinking differently.

if "isset" WEBHOOK_ALL_TITLE -> Add line and value
if "empty" WEBHOOK_ALL_TITLE -> Remove line

Yeah, this would work well. Sorry for the delay.

[jammsen]

If you like this project, please consider giving this repo and the Docker-Hub-Repo a Star.

[jammsen]

Hey @Jadiction - Should the line "content": WEBHOOK_ALL_TITLE, be removed if the ENV variable is set to empty/nothing?

[jammsen]

Also please use the Prefix of titles like i set it up for FR, that helps me to stay organized.

[Jadiction]

Hey @Jadiction - Should the line "content": WEBHOOK_ALL_TITLE, be removed if the ENV variable is set to empty/nothing?

Good question, certain webhooks may require the content to be specified (even if empty).
Depending on how you'd like it, I can think of 2 solutions:

1. If the ENV variable is specified, the line gets used (aka don't check if empty, only if set), otherwise exclude the line.
2. To play it safe and default it to "Status update," check for the ENV variable to be "NULL" to exclude it
   if ENV var is set & not "null", include line else if the ENV variable is not empty, content = ENV var, else content = "Status update"

I understand this makes it a little specific and would need some extra explaining in the documentation, but I believe it adds customizability.

[jammsen]

WEBHOOK_ALL_TITLE will never be not set, because it goes in the Dockerfile for default-sets and Backwards-Compability.
This makes things more complicated. But when i understand you correctly you would offer:

if "isset" WEBHOOK_ALL_TITLE -> Add line and value
if "empty" WEBHOOK_ALL_TITLE -> Remove line

Right? I think talking in pseudo-code is more easy to explain than very comlicated text 🤣

[jammsen]

@Jadiction Ping

[jammsen]

It doesnt really save space but it works:

Pushing it to develop and to latest later.

[jammsen]

See https://discord.com/channels/532141442731212810/1209949579185881139/1211257198869749800

[jammsen]

@Jadiction Please go ahead and test it and give me feedback.

[jammsen]

@Jadiction all good, happy to help!