docs: add Delete by Country section to visitors docs#55
Merged
Conversation
Document the Delete by Country feature, including step-by-step usage, bulk deletion support, and audit log fields. Add a corresponding row to the privacy controls table in privacy.md.
Reviewer's GuideAdds documentation for the new "Delete by Country" feature in visitors.md, detailing usage, bulk deletion behavior, and audit logging, and integrates it into the existing visitors documentation flow. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
gstraccini
Bot
added
the
☑️ auto-merge
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
WalkthroughAdds documentation for a GDPR "Delete by country" erasure feature: a new control entry, a visitors guide section describing selecting countries, confirmation with affected counts, single-transaction bulk deletion, and auditable deletion logs retained for 90 days; also updates the PHP example for programmatic usage. ChangesDelete by Country Feature Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![gstraccini[bot]](https://avatars.githubusercontent.com/in/480132?s=60&v=4){kind=small}
|
Failed to generate code suggestions for PR |
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Document the Delete by Country tool and its GDPR Article 17 workflow: targeted bulk deletion, confirmation dialog, and audit logging. Add `delete_by_country` and `delete_by_countries` API examples alongside existing `delete_ip` snippet.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/privacy.md (1)
60-60: ⚡ Quick winAdd a direct docs link for faster navigation.
Consider linking this row to the detailed “Deleting records by country” section so operators can jump straight to procedure/audit details.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/privacy.md` at line 60, Update the table row that reads "**Delete by country** | Visitors screen → Delete by Country | Bulk-removes all records..." to include an internal markdown anchor link pointing to the detailed "Deleting records by country" section so operators can jump directly to that section; modify the cell containing "Delete by Country" (or the whole row link) to use standard markdown link syntax with the section anchor (e.g., [Delete by Country](`#deleting-records-by-country`)) so the table entry navigates to the detailed procedure/audit details.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@docs/privacy.md`:
- Line 60: Update the table row that reads "**Delete by country** | Visitors
screen → Delete by Country | Bulk-removes all records..." to include an internal
markdown anchor link pointing to the detailed "Deleting records by country"
section so operators can jump directly to that section; modify the cell
containing "Delete by Country" (or the whole row link) to use standard markdown
link syntax with the section anchor (e.g., [Delete by
Country](`#deleting-records-by-country`)) so the table entry navigates to the
detailed procedure/audit details.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 5cf38df8-f717-40a3-9a8e-559ac3685664
📒 Files selected for processing (2)
docs/privacy.mddocs/visitors.md
Include the new GDPR erasure feature in the features table of the documentation. This update highlights the ability to bulk-delete visitor records by country, a feature that includes a confirmation dialog and audit log. This enhancement is important for compliance with privacy regulations and provides users with a critical tool for managing visitor data according to geographic privacy requirements.
|
Contributor
✅ WordPress Plugin Check Report
📊 ReportAll checks passed! No errors or warnings found. 🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check |
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
2 participants
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.
📑 Description
Document the Delete by Country feature, including step-by-step usage, bulk deletion support, and audit log fields. Add a corresponding row to the privacy controls table in privacy.md.
✅ Checks
☢️ Does this introduce a breaking change?
Summary by Sourcery
Document the new Delete by Country functionality for visitor records and align privacy documentation accordingly.
Documentation:
Summary by CodeRabbit