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(reports): ability to restrict reports server wide #1171

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JPBM135
Copy link
Contributor

@JPBM135 JPBM135 commented Jan 11, 2023

This PR adds a sub-command to the reports command with the ability to restrict reports server wide.

Note

This PR requires a command redeploy
This PR requires a migration under 1673411947-reports-restriction-level.js

Closes #1152

Config change

Command:

image

Updating levels:

image

Report examples

Restricted report:

image

Blocked report:

image

@vercel
Copy link

vercel bot commented Jan 11, 2023

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

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 11, 2023

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
yuudachi-report ⬜️ Ignored (Inspect) Visit Preview Sep 17, 2024 5:57am

Copy link
Member

@almostSouji almostSouji left a comment

Choose a reason for hiding this comment

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

I think giving the option to display a mod-defined message as well would be nice! So you can put "We are aware of the ongoing raid and are handling the situation as well as we can"

Shower thought:
Maybe this should introduce a level of confirmation before reports but still let them come through. That way we would inform users that we are aware of the ongoing situation, but still let them report general misconduct.

@JPBM135
Copy link
Contributor Author

JPBM135 commented Mar 8, 2023

Of course! This seems like an better approach than blocking completely, maybe add an option to warn or block?

@JPBM135 JPBM135 changed the title feat(reports): ability to disable reports server wide feat(reports): ability to restrict reports server wide Sep 17, 2024
@JPBM135 JPBM135 force-pushed the feat/reports-toggle branch from 20006bc to c9de5d7 Compare September 17, 2024 04:44
@JPBM135 JPBM135 requested a review from almostSouji September 17, 2024 04:45
@JPBM135 JPBM135 force-pushed the feat/reports-toggle branch from c9de5d7 to e921952 Compare September 17, 2024 05:50
@JPBM135 JPBM135 force-pushed the feat/reports-toggle branch from e921952 to c69e16d Compare September 17, 2024 05:57
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.

feat(reports): ability to suspend report creation
2 participants