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

A default embed helper function #30

Open
zeeshanok opened this issue Dec 24, 2020 · 4 comments
Open

A default embed helper function #30

zeeshanok opened this issue Dec 24, 2020 · 4 comments
Assignees
@vivekashok1221
Labels
area: utilities priority: high High priority type: feature A request for or implementation of a new feature

Comments

@zeeshanok
Copy link

Currently the process of making an embed is kinda annoying so a helper function can definitely make things faster.

I can implement this feature.
@gustavwilliam
Copy link
Member

This would be great to have. We could have different types of default embeds, like “categories.” For example, you could pass “error” as an argument and it would add the right emojis and colors for the error embed. This would help make the bot feel consistent and make the code more DRY.

We discussed this with @Arnav-2004 and Vyvy on the server earlier. This is a really helpful feature to have, so I'd love to see some action on this front.

@gustavwilliam gustavwilliam added area: utilities priority: high High priority type: feature A request for or implementation of a new feature labels Jan 2, 2021
@Arnav-2004
Copy link
Contributor

Yes this would be very helpful

This was referenced Jan 11, 2021
Merged
Merged
@vivekashok1221
Copy link
Member

I'll try to give this a shot.

@vivekashok1221 vivekashok1221 self-assigned this Jan 16, 2021
@gustavwilliam
Copy link
Member

Like master Yoda once said:

Do or do not, there is no try

;)

vivekashok1221 added a commit that referenced this issue Jan 17, 2021
@vivekashok1221
change in error emebed helper
2a3f4b7 
This method is to be removed after A default embed helper function #30
@vivekashok1221 vivekashok1221 mentioned this issue Jan 17, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vivekashok1221 vivekashok1221

Labels
area: utilities priority: high High priority type: feature A request for or implementation of a new feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@vivekashok1221 @gustavwilliam @zeeshanok @Arnav-2004