Eddiebot: Feat: Add custom embed colors to messages

Created on 7 Dec 2020  路  12Comments  路  Source: EddieJaoudeCommunity/EddieBot

Feature Request

The embeds made by Eddiebot are currently blue only and custom color embeds can be used to make embeds look more colorful and insightful.

Screenshots of current Embeds:

Image
For details regarding the discussion: https://github.com/EddieJaoudeCommunity/support/discussions/474
Idea raised by : @Vyvy-vi
Suggestions by @eddiejaoude :

  • standup: green
  • ... : amber
  • alexjs: red

Useful Resources:

Most helpful comment

I may have an idea for this new feature.
As you can see the defaultEmbed function does not have any parameters and we could add a color parameter as such:

-  defaultEmbed: () => {
-    return new MessageEmbed()
-      .setColor('#0099ff')
-      ...
+  defaultEmbed: (color: string) => {
+    return new MessageEmbed()
+      .setColor(color)
+      ...

Any thoughts on the matter?

Maybe, have a default parameter?

Like this:
ts defaultEmbed: (color: string = '#0099ff') => { return new MessageEmbed() .setColor(color) ...

All 12 comments

It's great having you contribute to this project

Feel free to raise an Issue! Welcome to the community :nerd_face:

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 馃 . Any questions let us know.

We should decide on colors for each command.
For example:

^bio -> amber
^coc ->
^help -> blue
^iam ->
^issue ->
^roles ->
^reminder ->
^standup -> green
^stats -> 
^tips ->
^support ->
alexjs-> red

Please add more colors if you have any idea because i am out of ideas.

hmm perhaps we could colour code them in groups according to functionality?

bio, roles, iam: colour 1 (personalisation)
support, issue, coc, tips: colour 2 (github/opensource) -> github-gray or some other colour
help embeds, reminder embeds: colou3 (utils)
alexjs, error_embeds: red (flags/negation)
system messages- green/red (logs)

(we could use some nice colour pallets from Coolors colour pallets)

Good idea grouping them by functionality.
I will try to find a solution for this and make a PR but it might take a couple of days because of college. 馃槃

I may have an idea for this new feature.
As you can see the defaultEmbed function does not have any parameters and we could add a color parameter as such:

-  defaultEmbed: () => {
-    return new MessageEmbed()
-      .setColor('#0099ff')
-      ...
+  defaultEmbed: (color: string) => {
+    return new MessageEmbed()
+      .setColor(color)
+      ...

Any thoughts on the matter?

I may have an idea for this new feature.
As you can see the defaultEmbed function does not have any parameters and we could add a color parameter as such:

-  defaultEmbed: () => {
-    return new MessageEmbed()
-      .setColor('#0099ff')
-      ...
+  defaultEmbed: (color: string) => {
+    return new MessageEmbed()
+      .setColor(color)
+      ...

Any thoughts on the matter?

Maybe, have a default parameter?

Like this:
ts defaultEmbed: (color: string = '#0099ff') => { return new MessageEmbed() .setColor(color) ...

This is coming along great 馃憤 . Awesome team work 馃挭

I think grouping the colours makes sense 馃憤

Yep really good teamwork 馃憤 .
So the default color is this blue #0099ff.
The color for alerts(such as alexjs, errors) could be this red #e84118.
The color for system messages could be this green #4cd137.
The color bio, roles, iam could be amber #ffc200.
The color for support, issue, coc, tips could be github-gray #ffffff.
The color for help embeds, reminder embeds could be purple#9c88ff.

If you have any other suggestions please post them. I'm not really good at color picking 馃構

New problem for this Issue.
I already posted the defaultEmbed() function, but there are some messages not being sent with this I think:

commandHandler/help.ts

commandHandler/bio.ts
I don't think the defaultEmbed() function is being used here, but I might be wrong.

We might have to add setColor there as well.

Share your thoughts please. 馃構 馃憤

we'll have to use .setColor()

The PR for this feature has been merged. Closing.

Thank you 馃憤

Was this page helpful?
0 / 5 - 0 ratings

Related issues

stemount picture stemount  路  11Comments

adityaraute picture adityaraute  路  18Comments

vinzvinci picture vinzvinci  路  13Comments

BOLT04 picture BOLT04  路  36Comments

Akshu-on-github picture Akshu-on-github  路  14Comments