Remove the docs refresh planning document - #622
Merged
Conversation
The working document served its purpose: every phase is executed, struck with a recorded reason, or absorbed. The only outstanding work it tracked — the three manual capture sets — moves to the capture README, including the previously unrecorded step-by-step IDE-plugins script. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
🟢 Ready to approve
The changes are limited to documentation/file removal, and the only feedback is minor path-clarity fixes in the README.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This PR cleans up the repository by removing the completed docs refresh planning document (per its own instruction) and leaves a small, actionable note in the screenshot-capture documentation about the remaining screenshots that still require manual capture.
Changes:
- Delete
REFRESH_PLAN.mdnow that the tracked work is complete. - Add a “Manual captures still owed” checklist to
screenshots/README.mdfor the remaining non-automatable screenshot sets.
File summaries
| File | Description |
|---|---|
| screenshots/README.md | Documents the remaining screenshot sets that must be captured manually and how to approach them. |
| REFRESH_PLAN.md | Removes the completed refresh planning document from the repo. |
Review details
Suppressed comments (1)
screenshots/README.md:80
faq/ide-plugins/is not an actual path in this repo; the docs live underen/faq/ide-plugins/andnl/faq/ide-plugins/. Using the real paths here will make it easier to find the current images and update them when the manual re-capture happens.
- **IDE plugins** (`faq/ide-plugins/`, 10 desktop-app images): the UI is
locale-independent, so capture ONE set and store it once (`images/ide-plugins/`),
referenced from both locales. One consistent OS and light IDE theme, latest
- Files reviewed: 2/2 changed files
- Comments generated: 1
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
Comment on lines
+75
to
+77
| - **Ufora / D2L** (`en|nl/guides/teachers/ufora/`, three images per locale, 2020 | ||
| vintage): external Brightspace UI; re-verify the whole flow while re-shooting — | ||
| the content tooling has changed since the shots were taken. |
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
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.
This pull request removes REFRESH_PLAN.md, per its own instruction to delete it when the refresh is done. Every phase it tracked is now executed (PRs #609-#621), struck with a recorded reason, or absorbed into restructured pages.