Skip to content

feat: Add Open button to spaced repetition question phase#421

Merged
rjroy merged 1 commit intomainfrom
feat/open-card
Jan 25, 2026
Merged

feat: Add Open button to spaced repetition question phase#421
rjroy merged 1 commit intomainfrom
feat/open-card

Conversation

@rjroy
Copy link
Copy Markdown
Owner

@rjroy rjroy commented Jan 25, 2026

Summary

  • Adds Open button to question phase that opens the card file for editing
  • Enables fixing time-sensitive questions (e.g., adding "As of Jan 20..." context) without leaving review
  • Revealed phase Open button still navigates to source file for context

Test plan

  • Start spaced repetition review with due cards
  • Verify Open button appears in question phase
  • Click Open and confirm it navigates to the card markdown file in Recall tab
  • Return to Ground tab and continue review
  • Reveal answer and verify Open button still opens source file (not card file)

🤖 Generated with Claude Code

Adds an Open button during the question phase that opens the card file
itself for editing. This enables users to fix questions that need
temporal context (e.g., adding "As of Jan 20..." to time-sensitive
questions) without leaving the review flow.

- Add card_file to DueCardSchema (path to card markdown file)
- Backend constructs card_file path from vault metadataPath
- Question phase: Open button navigates to card file for editing
- Revealed phase: Open button still navigates to source file for context

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 25, 2026

Codecov Report

❌ Patch coverage is 76.47059% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...end/src/components/home/SpacedRepetitionWidget.tsx 73.33% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@rjroy rjroy merged commit 75ad2ef into main Jan 25, 2026
2 checks passed
@rjroy rjroy deleted the feat/open-card branch January 25, 2026 16:26
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