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

Fix the fix to the PR template, emphasise the pre-approval list #26543

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

S34NW
Copy link
Member

@S34NW S34NW commented Aug 19, 2024

What Does This PR Do

Fixes the link to the pre-approval requirements.
Makes the requirement list stand out more with an important dialog.

Why It's Good For The Game

Our links need to work, they were broken in #26516

Images of changes

delwedd

Testing

Clicked the below link, looked at the text


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.

@S34NW S34NW added the Documentation This PR updates documentation either on GitHub, or in-code label Aug 19, 2024
@Burzah Burzah added this pull request to the merge queue Aug 19, 2024
@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting merge This PR is ready for merge label Aug 19, 2024
Merged via the queue into ParadiseSS13:master with commit 1ad1a89 Aug 19, 2024
11 checks passed
Hearstrings pushed a commit to Hearstrings/Paradise-SS220 that referenced this pull request Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting merge This PR is ready for merge Documentation This PR updates documentation either on GitHub, or in-code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants