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

feat(GuildChannelManager): add .addFollower() method #8567

Merged

Conversation

Syjalo
Copy link
Contributor

@Syjalo Syjalo commented Aug 28, 2022

Please describe the changes this PR makes and why it should be merged:
This PR adds GuildChannelManager#addFollower() method.

Status and versioning classification:

  • Code changes have been tested against the Discord API, or there are no code changes
  • I know how to update typings and have done so, or typings don't need updating
  • This PR changes the library's interface (methods or parameters added)

@vercel
Copy link

vercel bot commented Aug 28, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
discord-js ✅ Ready (Inspect) Visit Preview Oct 16, 2022 at 4:00PM (UTC)
1 Ignored Deployment
Name Status Preview Updated
discord-js-guide ⬜️ Ignored (Inspect) Oct 16, 2022 at 4:00PM (UTC)

Copy link
Member

@kyranet kyranet left a comment

Choose a reason for hiding this comment

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

The description may need wording changes, and I'm not entirely convinced by the method name, I feel like follow is better.

packages/discord.js/src/managers/GuildChannelManager.js Outdated Show resolved Hide resolved
packages/discord.js/src/managers/GuildChannelManager.js Outdated Show resolved Hide resolved
@Jiralite
Copy link
Member

Hey @Syjalo, are you still maintaining this pull request?

@vercel
Copy link

vercel bot commented Oct 13, 2022

@Syjalo is attempting to deploy a commit to the discordjs Team on Vercel.

A member of the Team first needs to authorize it.

Syjalo and others added 6 commits October 13, 2022 18:59
Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
Co-authored-by: A. Román <kyradiscord@gmail.com>
Co-authored-by: A. Román <kyradiscord@gmail.com>
@Syjalo Syjalo force-pushed the feat/add-GuildChannelManager#addFollower() branch from e898e94 to 843986e Compare October 13, 2022 15:59
@Jiralite Jiralite requested a review from kyranet October 13, 2022 17:11
@CitTheDev
Copy link
Contributor

I suggest you add this method to the GuildChannel class and add typings to make sure the channel type is GuildAnnouncement.

@Jiralite
Copy link
Member

It shouldn't be added to that class as not all guild channels may do that. Only one type of guild channel may do that, and its functionality already exists.

@kodiakhq kodiakhq bot merged commit caeb1cb into discordjs:main Oct 16, 2022
@Syjalo Syjalo deleted the feat/add-GuildChannelManager#addFollower() branch January 26, 2023 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

7 participants