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

Add Forum Pinner module #5

Merged
merged 2 commits into from
Mar 20, 2024
Merged

Add Forum Pinner module #5

merged 2 commits into from
Mar 20, 2024

Conversation

Ale32bit
Copy link
Contributor

This module automatically pins the first message of newly created forum posts.

It also adds the /pin command that pins the first message it fetches (probably needs rework?)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for your pull-request! It looks like this is your first PR to AuroraBot. That is awesome :D ! Soon, the maintainers should take a look at your work. In the meantime, to increase chances of having your PR accepted, please make sure that the PR contains an explaination of what your PR contains, make sure your code is formatted properly, make sure that everything is working and tested with your own Discord instance and make sure that there is little room for abuse by users. Thanks!

modules/forum_pinner.js Show resolved Hide resolved
modules/forum_pinner.js Show resolved Hide resolved
@Lemmmy
Copy link
Collaborator

Lemmmy commented Jan 13, 2024

Mainly not sure what the intended use of the /pin command is. Is it staff only? If not, I'm wondering if it's worth being a separate PR so that it can be evaluated independently of the more important functionality here (automatic pinning).

@Ale32bit
Copy link
Contributor Author

The pin command is useful for threads that were made prior to this module running, it is of course only executable by members with the MANAGE_MESSAGES permission.

@Lemmmy
Copy link
Collaborator

Lemmmy commented Jan 13, 2024

If the command is staff-only then it's all good in the current state 👍

@Lemmmy Lemmmy merged commit d8f76a8 into Wendelstein7:main Mar 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants