Ah ha, thanks for that! I definitely see the same behavior.
Sending mention notifications through a post edit is an important function that I know I and others use from time to time, so we want that to stay, but the way it’s working with groups feels off.
My opinion at this point would be to only notify a group once per post. Users added to a group after an initial group mention was made should not receive a group mention notification as a result of subsequent post edits. The JIT education banner about notifying the group should not display during the editing of a post if there are no new group mentions since the post was last saved.
If a group that was not previously mentioned is mentioned in an edit, they should be notified. That’s how it works with user mentions. We should also see the relevant educational banner if there is a newly mentioned group.
I’m thinking we can just add some conditional logic based on the existence of a record in the group_mentions table. We create a record at the time of saving a post.
What do you think @codinghorror?