Skip to content

directory: show knocking status in listings, trigger re-review on knocking change#6695

Merged
epoberezkin merged 4 commits intomasterfrom
nd/improve-directory
Mar 24, 2026
Merged

directory: show knocking status in listings, trigger re-review on knocking change#6695
epoberezkin merged 4 commits intomasterfrom
nd/improve-directory

Conversation

@Narasimha-sc
Copy link
Copy Markdown
Collaborator

Summary

  • Fix sameProfile to include memberAdmission so toggling knocking on a listed group triggers re-review and regenerates the web listing JSON with the updated admission field
  • Show "Knocking: enabled" in bot search results (/search, /next) for groups with review = MCAll
  • Show "Knocking: enabled" in bot /list output between status and commands lines
  • Show "Knocking: enabled" in web directory UI under the member count

Test plan

  • Toggle knocking on an active listed group; verify bot sends "updated, hidden until approved" message and notifies admins
  • Search for a group with knocking enabled; verify "Knocking: enabled" appears in the result
  • Send /list; verify groups with knocking show "Knocking: enabled" line
  • After approving a group with knocking, check data/listing.json and confirm "admission": {"review": "all"} is present
  • Load the directory page and verify groups with knocking show "Knocking: enabled" under the member count

🤖 Generated with Claude Code

…cking change

- fix sameProfile to include memberAdmission so toggling knocking triggers re-review and regenerates the web listing JSON
- show "Knocking: enabled" in bot search results and /list output
- show "Knocking: enabled" in web directory UI

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@epoberezkin epoberezkin merged commit 427ce1b into master Mar 24, 2026
8 of 11 checks passed
@epoberezkin epoberezkin deleted the nd/improve-directory branch March 24, 2026 20:02
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants