Xeio / WanderLost

Lost Ark Wandering Merchant Tracker
MIT License
106 stars 36 forks source link

Discord bot notifications missing details #209

Closed ann-one closed 1 year ago

ann-one commented 1 year ago

Thank you for adding Discord bot notifications! Can you possibly add location and card details into the text of the Discord bot notifications to match the format of the web push notifications? Right now the text body for each notification just says "Lost Merchants card notification" without any details, so it requires clicking through to the message in Discord to figure out which card and spawn location.

Xeio commented 1 year ago

Eh? It should look like this

image

Both the desktop and mobile interfaces look like that for me (unless iOS behaves different than Android does?)

ann-one commented 1 year ago

I'm specifically referring to the text body of the Android system notification. If you simplify the way the DM text is formatted then all the relevant information can also show up in the body of the system notification. So rather than prefacing the message with "Lost Merchants card notification" you could use the text "Osphere Card - Rowen - The Wolflands".

image

vs

image

Xeio commented 1 year ago

Ah, that's a bit trickier since Discord controls the formatting of the actual notification. Seems like it may just be puling from the text of the link header so I could change that.

ann-one commented 1 year ago

It looks like you updated the message format and details shows up in the notification now! Tysm!