Skip to content

Add required internal note field and fix redirect after slug change#604

Merged
duckduckgrayduck merged 2 commits intomasterfrom
issue_535_note_field
Mar 4, 2026
Merged

Add required internal note field and fix redirect after slug change#604
duckduckgrayduck merged 2 commits intomasterfrom
issue_535_note_field

Conversation

@duckduckgrayduck
Copy link
Copy Markdown
Contributor

Currently when a staff member accepts a slug change the redirect to the old org url fails. This one pulls the new slug every time, so it will correctly redirect. This PR also closes #535
The formatting in the template may need work.

@allanlasser allanlasser temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 16:38 Inactive
@duckduckgrayduck duckduckgrayduck temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 16:48 Inactive
@duckduckgrayduck duckduckgrayduck temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 16:52 Inactive
@duckduckgrayduck duckduckgrayduck temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 16:56 Inactive
@duckduckgrayduck duckduckgrayduck temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 17:07 Inactive
@duckduckgrayduck duckduckgrayduck temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 17:46 Inactive
@duckduckgrayduck duckduckgrayduck temporarily deployed to squarelet-pi-issue-535--qrh5yn February 26, 2026 18:03 Inactive
Copy link
Copy Markdown
Member

@allanlasser allanlasser left a comment

Choose a reason for hiding this comment

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

Everything looks good to me! I updated the styling on the review form:

Image

@duckduckgrayduck duckduckgrayduck merged commit 74b535c into master Mar 4, 2026
6 checks passed
@duckduckgrayduck duckduckgrayduck deleted the issue_535_note_field branch March 9, 2026 20:12
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.

Add a note field to capture reasons for rejected change requests

3 participants