Skip to content

Send notification meta checkbox #366

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

marclucraft
Copy link
Contributor

@marclucraft marclucraft commented Jul 24, 2025

#348 visually unchecked the box on post, but a page refresh results in the checkbox being checked again.


This change is Reviewable

@marclucraft marclucraft requested a review from sherwinski July 24, 2025 13:24
@sherwinski
Copy link
Contributor

My understanding was that the current behavior was what was always intended, i.e. so long as the "Send notification" setting is enabled from the admin page then the box should be enabled, even after refresh. Happy to be wrong on that but figured I'd at least double check.

@rfischmann
Copy link

My understanding was that the current behavior was what was always intended, i.e. so long as the "Send notification" setting is enabled from the admin page then the box should be enabled, even after refresh. Happy to be wrong on that but figured I'd at least double check.

If I may share my opinion, at least based on our work at MacMagazine, it would be a disaster if the checkbox was enabled by default even on posts that are already published. We're a multi-person team and we constantly edit posts from each other to add new information, fix mistakes, etc. It would be a nightmare for us to always have to remember to uncheck that option. We only fire new pushes for previous posts when the edit is very relevant and we want to let our audience know that the post was edited.

That's one of the reasons we're still on plugin version 2.4.4. It works exactly that way.

@sherwinski
Copy link
Contributor

@rfischmann That makes sense, thanks for the feedback!

@rfischmann
Copy link

Hey guys, is there any ETA for a plugin update with this fix? Thanks!

@sherwinski
Copy link
Contributor

Hey @rfischmann, I'm just wrapping up a couple of other tasks before moving on to this. We will you keep you updated here with any changes.

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.

3 participants