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

Updates the issue report template to hopefully better draw attention to what may be an exploit #28473

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

1080pCat
Copy link
Contributor

What Does This PR Do

Updates the issue form template, hopefully to attract more attention to if someone may be reporting a exploit in the public by having two check boxes with text detailing the two major factors to an exploit

Why It's Good For The Game

Mistakes happen or sometimes some people define exploits differently to how we do, hopefully making it so you need to check two boxes with text next to them detailing what we define as an exploit will help.

Images of changes

image

Testing

Im going to be completely honest, im not totally sure how to test this outside of looking at the file preview on github, not without doing some merge stuff with my local branch that i don't feel overly confidant in being able to undo after the fact. However as far as i can tell it works


Declaration

  • I confirm that I either do not require pre-approval for this PR, or I have obtained such approval and have included a screenshot to demonstrate this below.

Changelog

NPFC

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Feb 21, 2025
@Contrabang Contrabang added the Documentation This PR updates documentation either on GitHub, or in-code label Feb 21, 2025
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting review This PR is awaiting review from the review team and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Feb 21, 2025
Co-authored-by: Nathan Winters <[email protected]>
Signed-off-by: 1080pCat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting review This PR is awaiting review from the review team Documentation This PR updates documentation either on GitHub, or in-code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants