Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Add Emojis as Option in userinfo #47

Merged
merged 7 commits into from Aug 24, 2022
Merged

Add Emojis as Option in userinfo #47

merged 7 commits into from Aug 24, 2022

Conversation

UltimateNyn
Copy link
Collaborator

@UltimateNyn UltimateNyn commented Aug 5, 2022 •
edited

This adds the Option to also show Emoji stats in the userinfo commands. Before it gets merged the following things should be improved:

  • sort the Emojis by count
  • format the Emojis in a better way

@Kirottu Kirottu marked this pull request as draft August 8, 2022 10:23
@UltimateNyn
split contextmenu in own command
7812c97 
to do this the userinfo module was created
@UltimateNyn UltimateNyn force-pushed the userinfo-extended branch 3 times, most recently from c31db74 to cb14493 Compare August 12, 2022 21:10
@UltimateNyn UltimateNyn changed the title [Do not merge] Add Emojis as Option in userinfo Add Emojis as Option in userinfo Aug 13, 2022
@UltimateNyn UltimateNyn marked this pull request as ready for review August 13, 2022 12:01
Kirottu
Kirottu reviewed Aug 19, 2022
View reviewed changes
Copy link
Contributor

@Kirottu Kirottu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be fine, the older PR should probably be merged first though. As soon as that is done this can be merged as well

@Kirottu Kirottu merged commit bc1857a into TheTerminalCafe:develop Aug 24, 2022
@UltimateNyn UltimateNyn deleted the userinfo-extended branch August 31, 2023 13:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Kirottu Kirottu Kirottu left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@UltimateNyn @Kirottu