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.

Sign up for GitHub

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

Jump to bottom

Please provide a helper that supports changing to escape as \ n in the template #217

Open
goodmangll opened this issue Feb 22, 2024 · 2 comments · May be fixed by #306
Open

Please provide a helper that supports changing to escape as \ n in the template #217

goodmangll opened this issue Feb 22, 2024 · 2 comments · May be fixed by #306

Comments

@goodmangll
Copy link

Due to some variables having line breaks, the final result does not conform to a JSON. Please take a look at the example below.

my template

image

final result

截图_选择区域_20240222142459
stoggi added a commit to stoggi/jellyfin-plugin-webhook that referenced this issue Mar 30, 2024
@stoggi
Attempt to fix JSON encoded strings for the discord webhooks. see jel…
daaa281 
…lyfin#217
@stoggi stoggi mentioned this issue Mar 30, 2024
Open
@seanmcbroom
Copy link
Contributor

Could it be because the overview is html escaped?
https://handlebarsjs.com/guide/#html-escaping

@Bpendragon
Copy link

I'm also running into this issue, the description/overview that the webhook is trying to send is malformed and Discord throws an error. This causes items to be missed and notifications never sent (or at least not sent in a way that is visible on discord)

@Bpendragon Bpendragon linked a pull request Nov 18, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@Bpendragon @goodmangll @seanmcbroom