docs(admin): say what pending state actually costs - #12
Merged
Conversation
"This can cause data inaccuracies. Use with caution." named no consequence, so an admin had nothing to weigh. Replaced with the real cost: a Pending repeater doesn't associate coverage, so pings heard while it waits stay unlinked — and approving it later doesn't backfill them, which is the part that bites. Points at Reassociate Repeater for recovering them. Matching change made to both pending help texts in the panel (dev 24aa606). Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KojjSrgSQQJW7Z8b3eakEA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
"Can cause data inaccuracies" names no consequence, so an admin has nothing to weigh when deciding whether to turn the setting on.
Replaced with what actually happens:
The non-retroactive part is what bites and wasn't stated anywhere: approval is just a status change, so pings that arrived during the Pending window stay unlinked unless you run the reassociate tool.
Matching change made to both pending help texts in the panel — group and single-region both carried the same vague wording (
MeshMapper_Server24aa606, ondev).🤖 Generated with Claude Code
https://claude.ai/code/session_01KojjSrgSQQJW7Z8b3eakEA