Skip to content

Upgrade the dashboard to Vite 8 and TypeScript 6 and apply bug fixes and enhancements - #96

Open
alfawal wants to merge 2 commits into
TkTech:26_uxfrom
alfawal:26_ux-vite8
Open

alfawal wants to merge 2 commits into
TkTech:26_uxfrom
alfawal:26_ux-vite8

Conversation

@alfawal

@alfawal alfawal commented Aug 11, 2026

Copy link
Copy Markdown

I've tried the #58 dashboard and found a few bugs and did a few improvements along the way:

  • Upgraded to Vite 8 and TypeScript 6 (also updated the engines to Vite's floor), I wanted to upgrade to TypeScript 7 but typescript-eslint doesn't support it yet.
  • Fixed a login failure due to cross-origin, fetches still sent credentials: 'include' from the cookie-auth days, the browser rejected when the server answered with Access-Control-Allow-Origin: *. So since the auth is token based, i've removed it.
  • Fixed editing a queue returns 500: useEntityForm used its Zod schema only as a validator, so the schema's transforms never ran and raw form strings went straight to Postgres. Now it parses on submit.
  • Fixed a 500 error on PATCH even after a successful write: declare_ex returned state as a raw string, crashing Queue.pack(), and json_dumps couldn't serialize sets which killed the websocket stream event.
  • Deleting a queue from its detail page redirected to "Queue not found"; it now redirects to the queue list.
  • Purging a job left the drawer open ~5s: purge invalidated the deleted job's detail query, which refetched, 404'd, and went through React Query's retry backoff before anything closed. Purge now drops the dead query and the drawer (or details page) closes immediately.

@alfawal

alfawal commented Aug 11, 2026

Copy link
Copy Markdown
Author

Out of curiosity, why don't we use a non-locking baseline component library and iterate on it? like shadcn or something, I think there is room for UI/UX improvements, if you're interested in this kind of improvements let me know I can pitch a prototype for whats in my mind.

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.34%. Comparing base (91c03e0) to head (67559cf).
⚠️ Report is 24 commits behind head on 26_ux.

Files with missing lines Patch % Lines
chancy/utils.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            26_ux      #96      +/-   ##
==========================================
+ Coverage   67.42%   68.34%   +0.91%     
==========================================
  Files          62       62              
  Lines        3635     3680      +45     
==========================================
+ Hits         2451     2515      +64     
+ Misses       1184     1165      -19     
Flag Coverage Δ
unittests 68.34% <0.00%> (+0.91%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

This branch has not been deployed

No deployments
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