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 a report feature for bots, server and templates #345

Merged
merged 7 commits into from
Mar 2, 2025

Conversation

carolinaisslaying
Copy link
Member

Description

This adds a report feature to bots, servers, and templates. Resolves #159.

Changes Summary

  • Adds a new property in settings.json as settings.emoji.report.
  • Add a report flow to bots, servers and templates that is visible to logged in users except the bot owner.
  • Report flow has a set of default reasons plus an other, which on click opens the second part of the flow allowing for additional information to be submitted.
  • On submit, it sends a message to the site-alerts channel and closes the modal for the user on response with a toast depicting whether it was successful or not.

Screenshots / Demo (if applicable)

Toggle this collapsible list to see images.

Hero of bot page showing report button placement

image

Recording showing report flow

Screen.Recording.2025-03-02.at.3.58.38.PM.mov

Report logged in #site-alerts

image

Type of Change

  • Feature enhancement
  • Bug fixes
  • Refactor
  • Breaking change
  • Performance improvement/reducing time complexity

Confirmation Checklist

  • I have ensured that my code follows the project's style guidelines (pnpm run pretty).
  • I have performed a self-review of my code and corrected any issues.
  • I have tested the changes to verify they work as intended.
  • I have confirmed that my changes do not introduce new security vulnerabilities.
  • I have documented and communicated any breaking changes appropriately.

@carolinaisslaying carolinaisslaying added the feature enhancement Issue/PR related to a feature implementation, etc. label Mar 2, 2025
@carolinaisslaying carolinaisslaying added this to the 5.7.0 milestone Mar 2, 2025
carolinaisslaying added a commit to discordextremelist/i18n that referenced this pull request Mar 2, 2025
@carolinaisslaying carolinaisslaying merged commit ab384ae into main Mar 2, 2025
4 checks passed
@carolinaisslaying carolinaisslaying deleted the report-feature branch March 2, 2025 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature enhancement Issue/PR related to a feature implementation, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

report a bot
1 participant