pascalla / clan-chat-webhook

BSD 2-Clause "Simplified" License
3 stars 1 forks source link

Clan Announcements #11

Closed Osamu01 closed 10 months ago

Osamu01 commented 11 months ago

I believe putting announcements such as level ups, etc should be formatted in bold or something to easier tell. that or you could put something like this in front of it. 800px-Dungeon_entrance_logo

pascalla commented 10 months ago

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

image

image

Ryglus commented 10 months ago

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

This new feature is broken since you cant disable it, i think that on server level you dont have correct naming as the name that is being checked/required.

ai this makes it so you can really update any change at all.

image

image

pascalla commented 10 months ago

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

This new feature is broken since you cant disable it, i think that on server level you dont have correct naming as the name that is being checked/required.

ai this makes it so you can really update any change at all.

image

image

Just pushed a fix, just a typo between clan_icons and clain_icons in validation logic. Let me know if this fixes it :)

Ryglus commented 10 months ago

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

This new feature is broken since you cant disable it, i think that on server level you dont have correct naming as the name that is being checked/required. ai this makes it so you can really update any change at all. image image

Just pushed a fix, just a typo between clan_icons and clain_icons in validation logic. Let me know if this fixes it :)

Yep, perfect, fixed, thank you

Osamu01 commented 10 months ago

Just got wind of this, how exciting! I will let you know if I have further suggestions.