test: verify multi-chart management implementation
- Verified ChartSelector uses theme-aware classes - Tested full workflow: upload, chart creation, switching, annotation scoping - Verified migration creates default chart with existing data - Tested chart deletion with cascade delete of candles/annotations - Tested edge cases: duplicate filenames, deleting last chart, upload with no charts - Build passes with no type errors
This commit is contained in:
parent
b9771fe89f
commit
7cb308788e
2 changed files with 8 additions and 8 deletions
|
|
@ -44,13 +44,13 @@
|
|||
|
||||
## 7. UI Polish
|
||||
|
||||
- [ ] 7.1 Restyle "Manage Annotation Types" link — replace `text-blue-600 hover:text-blue-800 underline` with `text-muted-foreground hover:text-foreground` and remove underline
|
||||
- [ ] 7.2 Verify chart selector and all components render correctly in both light and dark themes
|
||||
- [x] 7.1 Restyle "Manage Annotation Types" link — replace `text-blue-600 hover:text-blue-800 underline` with `text-muted-foreground hover:text-foreground` and remove underline
|
||||
- [x] 7.2 Verify chart selector and all components render correctly in both light and dark themes
|
||||
|
||||
## 8. Testing & Verification
|
||||
|
||||
- [ ] 8.1 Test full workflow: upload CSV -> chart created -> switch charts -> annotations scoped correctly
|
||||
- [ ] 8.2 Test migration: existing database with candles/annotations migrates to default chart
|
||||
- [ ] 8.3 Test chart deletion: candles and annotations cascade-deleted
|
||||
- [ ] 8.4 Test edge cases: upload duplicate filename, delete last chart, upload with no charts existing
|
||||
- [ ] 8.5 Run build (`npm run build`) and fix any type errors
|
||||
- [x] 8.1 Test full workflow: upload CSV -> chart created -> switch charts -> annotations scoped correctly
|
||||
- [x] 8.2 Test migration: existing database with candles/annotations migrates to default chart
|
||||
- [x] 8.3 Test chart deletion: candles and annotations cascade-deleted
|
||||
- [x] 8.4 Test edge cases: upload duplicate filename, delete last chart, upload with no charts existing
|
||||
- [x] 8.5 Run build (`npm run build`) and fix any type errors
|
||||
|
|
|
|||
|
|
@ -163,7 +163,7 @@ export default function Home() {
|
|||
<p className="text-sm text-muted-foreground mt-1">Chart annotation tool</p>
|
||||
<a
|
||||
href="/annotation-types"
|
||||
className="mt-3 inline-block text-sm text-blue-600 hover:text-blue-800 underline"
|
||||
className="mt-3 inline-block text-sm text-muted-foreground hover:text-foreground"
|
||||
>
|
||||
Manage Annotation Types
|
||||
</a>
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue