Skip to content

fix: web todo render issue#38

Merged
cnjack merged 1 commit intomainfrom
feat/web-todo-panel
Apr 24, 2026
Merged

fix: web todo render issue#38
cnjack merged 1 commit intomainfrom
feat/web-todo-panel

Conversation

@cnjack
Copy link
Copy Markdown
Owner

@cnjack cnjack commented Apr 24, 2026

Summary by CodeRabbit

  • New Features

    • Added support for "cancelled" status in todo items.
    • Updated visual indicators for different todo statuses with improved UI styling.
  • Updates

    • Improved todo status handling with new status categories: pending, in progress, completed, and cancelled.
    • Strike-through styling now applies to both completed and cancelled items.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 98e29a81-a337-42a6-9622-3f57552f630c

📥 Commits

Reviewing files that changed from the base of the PR and between f3c3fa5 and bb70e10.

📒 Files selected for processing (3)
  • web/src/components/TodoPanel.vue
  • web/src/stores/chat.ts
  • web/src/types/api.ts

📝 Walkthrough

Walkthrough

Field names in the TodoItem interface were renamed from uppercase to lowercase (IDid, Titletitle, Statusstatus). Status enum values were updated from 'not-started' | 'in-progress' | 'completed' to 'pending' | 'in_progress' | 'completed' | 'cancelled'. All component and store references were updated accordingly.

Changes

Cohort / File(s) Summary
Type Definitions
web/src/types/api.ts
Updated TodoItem interface: renamed fields to lowercase (id, title, status) and changed status enum to include 'pending' (replacing 'not-started'), 'in_progress' (replacing 'in-progress'), 'completed', and new 'cancelled'.
Component & Store Updates
web/src/components/TodoPanel.vue, web/src/stores/chat.ts
Refactored template expressions and computed logic to use lowercase field names and status values; updated conditional rendering and class bindings to match new status enum values.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hop hop, the fields are now so neat,
Lowercase letters, a naming retreat!
From in-progress dashes gone,
To cancelled marks, the old is withdrawn,
TodoItems spring forth with cleaner names,
Our little carrots dance in refactor flames! 🥕

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/web-todo-panel

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cnjack cnjack merged commit 5b6e492 into main Apr 24, 2026
1 check was pending
@cnjack cnjack deleted the feat/web-todo-panel branch April 24, 2026 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant