Skip to content

Conversation

@Samq64
Copy link
Member

@Samq64 Samq64 commented Aug 24, 2025

Resolves #8517

Changes

Checks the success message to ensure it's not an edit and run on topics instead of newPostScreens since it doesn't need those pages anymore.

Tests

Tested by editing a post.

@Samq64 Samq64 added the scope: addon Related to one or multiple addons label Aug 24, 2025
@CST1229
Copy link
Member

CST1229 commented Aug 24, 2025

does topic creation have the 60-second rule? (because if so then the countdown needs to appear there)

@mxmou
Copy link
Member

mxmou commented Aug 24, 2025

Technically, the notices such as "Reply saved." can appear on any page because of the way Django's notice system works. The addon already works fine almost all the time, but a more reliable implementation would be to run the userscript on all forum pages (in case the notice appears on a different page) and compare the message with those for creating a topic and adding a reply (to exclude signature updates, for example).

@Samq64 Samq64 requested a review from mxmou September 23, 2025 11:52
@DNin01 DNin01 added this to the v1.46.0 milestone Dec 7, 2025
@DNin01 DNin01 modified the milestones: v1.46.0, v1.45.0 Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: addon Related to one or multiple addons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

forum-post-countdown: Editing a post counts as creating one

4 participants