Repository navigation
CPD tidy-up: one End of year page, Job applications to Admin, More menu down to seven - #3460
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
…ion box after main fields Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
🏁 CodeAnt Quality Gate ResultsCommit: ✅ Overall Status: PASSEDQuality Gate Details
|
Merging this PR will not alter performance
|
Report becomes End of year: what is missing, evidence to tidy, hand it in. Evidence, Export and AMA CPD Home leave the More menu and stay as the pages each step opens. Saved favourites for moved pages open their new home. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
🤖 CodeAnt AI — Review Status
|
Thanks for using CodeAnt! 🎉We're free for open-source projects. if you're enjoying it, help us grow by sharing. Share on X · |
There was a problem hiding this comment.
BigSimmo has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
Claude encountered an error after 2s —— View job I'll analyze this and get back to you. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
PR Summary by QodoConsolidate CPD year-end tasks and move job applications to Admin
AI Description
Diagram
High-Level Assessment
Files changed (48)
|
…what it holds Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9045dd50f0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Code Review by Qodo
No findings for this group No findings for this group No findings for this group Resolved findings 1.
|
Qodo Fixer✅ Committed (5) · ☑ Fixed (5) Commits pushed directly to this PR — no separate fix PR opened. Process — 5 fixed
|
…dence state, test moved favourites - Job applications search results now sit under Admin, like its reminders. - Evidence keeps 'No activities logged' when the year has none, filters or not. - Tests: moved favourites open, unpin and remove by their new key; exactly three End of year steps. - Refresh the repo-awareness snapshot for the moved routes. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
|
Review findings addressed (commits 2af6e9b, 9ddcb25, e98f157, 5673d95 and a22c095):
Generated by Claude Code |
There was a problem hiding this comment.
BigSimmo has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
Code review by qodo was updated up to the latest commit a22c095 |
Qodo FixerFound 5 findings, but none include a fix that can be applied automatically. |
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a22c09501f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
|
Code review by qodo was updated up to the latest commit 9dac5ef |
…er CPD menu Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
Code Review ✅ Approved 2 closed / 2 findings🟡 Medium risk · Moving Job applications into Admin and consolidating CPD year-end pages changes routes, which could make saved plans or evidence actions harder to reach. Consolidates CPD year-end into one three-step End of year page, moves Job applications and its CV to Admin at
✅ 2 closed✅ Edge Case: Unchecked view hides filter chips but still obeys URL filters
✅ Bug: Moved favourites can't be removed, pinned or recorded as opened
Review coverage🤖 Auto-approval Approved 📋 Rules No rules evaluated 🧪 Functional validation Not enabled · Set up OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
|
Code review by qodo was updated up to the latest commit 0951a77 |
User description
Requested by Josh · project thread
Summary
Before: CPD had five menu pages for the end of the year (Report, Evidence, Export, AMA CPD Home, Annual summary), each repeating the activity list. Evidence showed a green "None known to need evidence" tick and then listed every activity as "Not counted". AMA CPD Home opened with a large "Format to be confirmed" card. Job applications sat in CPD. The More menu had 15 entries, including Customise and a Set up entry that opened the same page as Targets. The "replaces a missed session" tick box was the first thing on Log an activity.
After: Report becomes End of year, one page in three numbered steps: what is missing, evidence to tidy (with Attach on each activity lacking a certificate), and hand it in (save, copy to your CPD home, close the year). Evidence, Export and AMA CPD Home leave the menu and stay as the detail pages each step opens. The CPD More menu is down to seven. Evidence says plainly when nothing was checked yet. AMA CPD Home puts its caveat in one line. Job applications and its CV live at
/admin/applicationsin Admin's menu (old addresses redirect). The missed-session box sits after the main fields. Saved favourites for moved pages open their new home.How: Wording, layout and navigation only, plus the route move. No data, clinical wording or sign-off changes. From the 11 Oct usefulness and layout audit; Josh approved the End of year mockup.
Areas touched: Personal practice, App experience, Design system, Data platform and access, Knowledge and records
Verification
Test Files 95,1215tests, one weight rule fixed thencme-type-weight4 passed,cme-year-check17 passed),tsc --noEmitclean, ESLint clean on changed files, phone screenshot of End of year checked against the mockup. Browser suites left to CI.Risk and rollout
🤖 Generated with Claude Code
https://claude.ai/code/session_01JZNXNqu4ZWvQwW613xMf1P
CodeAnt-AI Description
Streamline CPD year-end review and move job applications to Admin
What Changed
Impact
✅ One guided path from CPD checks to year closure✅ Clearer evidence status when certificates have not been checked✅ Job applications and CV available from Admin💡 Usage Guide
Checking Your Pull Request
Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.
Talking to CodeAnt AI
Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:
This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.
Example
Preserve Org Learnings with CodeAnt
You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:
This helps CodeAnt AI learn and adapt to your team's coding style and standards.
Example
Retrigger review
Ask CodeAnt AI to review the PR again, by typing:
Check Your Repository Health
To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.