Custom Screenshot Titles

[mmekker]

Checklist

• I've searched the issues and pull requests for similar looking suggestions.
• I've checked the Unreleased section of the changelog for newly added features that sound like my suggestion.

Describe your Suggestion

Add the option for custom screenshot file names that use metadata such as playerName, NotificationType, and itemName.

Reasoning

Having multiple people in the same discord notification channel makes it impossible to see who the notification is from and what they did from the pop-up notification.

[pajlada]

Are you referring to the filename (e.g. xpImage.png) that's sent along to Discord with the webhook?

[mmekker]

Yes, the filename that is sent along to Discord with the webhook ends up being used as the notification text for push notifications.

[samrenfrew]

Came here to raise this. When a notification includes a screenshot, the notification just contains the image name e.g "lootImage.png" or "deathImage.png". Noticed for notifications where a screenshot does not include a screenshot, the notification text is correct and includes the players name and action.

Guessing this is a limitation of the Discord rich embed?

[pajlada]

I get the feeling that this is a limitation of Discord's embed and/or notification system. Just changing the filename could be one way to make it slightly better, but ultimately the issue would still be here.

If someone who knows more about Discord embeds know if we're doing some wrong, or if there's another "hidden" field we can add to use for notifications, please let us know

[samrenfrew]

@pajlada My cursory googling/AI-ing leads me to believe it's a Discord Mobile issue, where priority is given to the attached file rather than the embed itself. Whilst the image is an attachment to the embed, because you're adding both a payload_json and a file in the MultiPartForm, mobile seems to prioritize the file attachment for the notification. Not sure there is a workaround at this point!