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.

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

WIP: Group DMs #27

Closed
wants to merge 7 commits into from
Closed

WIP: Group DMs #27

wants to merge 7 commits into from

Conversation

Maks-s
Copy link
Contributor

@Maks-s Maks-s commented Jan 13, 2021

Add group DMs back !

There's still a lot of work to do, but as of now it works

TODO:

  • Group DM user list
  • Refactor DMButton, GDMButton, UserButton
  • Update group DMs' name when a recipient update their name
  • Find how discord get the default group DM's icon
  • Edit Group DM's properties

As a side-note, this should be a Draft PR but I tripped on my keyboard 🤦‍♂️

style: some files got linted
@ruslang02 ruslang02 added the enhancement New feature or request label Jan 13, 2021
@ruslang02 ruslang02 linked an issue Jan 13, 2021 that may be closed by this pull request
Ruslan Garifullin and others added 6 commits January 13, 2021 23:32
refactor(GDMButton): Use name instead of getName/setName
refactor(DMChannel): Move DMChannel to BaseUserButton
…use relationships aren't implemented

feat(GroupDM/Titlebar): add base for editing group DM's name in titlebar
feat(GroupDMChannel): add delete() function
fix(discord.js): fix error in Util.cleanContent
feat(GroupDM): add context menu
feat(GroupDM): rename group DM in titlebar
fix(GroupDMChannel): mute option
fix(Actions): error when channel wasn't in a guild
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Group DMs support
2 participants