Skip to content

Frontend optimizations #470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
e32wong wants to merge 32 commits into
base: main
Choose a base branch
from
Draft

Frontend optimizations #470

e32wong wants to merge 32 commits into from

Conversation

@e32wong
Copy link
Contributor

@e32wong e32wong commented Jan 18, 2026 •
edited
Loading

This PR will be rebased against the main branch once it is ready for review as it relies on a few compilation fixes from the other pull request (#457).

Changes:

  • Removed manual Vite chunking
    • The reason is that in profiling tests, we discovered that it is degrading the loading performance rather than enhancing it. In a 4G test with cache disabled, we are loading at 15.3 seconds (manual chunking disabled) vs 16 seconds (manual chunking optimizations enabled).
  • Unified the icon library to Iconify
    • Repeated tests showed a speedup of 0.3 seconds on a 4G test with cache disabled. Observing a drop from 12.9 seconds to 12.6 seconds.

e32wong added 30 commits January 15, 2026 18:06
@e32wong
Create API routes for the edit feature
6bc212c
@e32wong
Frontend integration with new API
c080614
@e32wong
Fix rebase errors
5d89da3
@e32wong
Fix import dependencies
4f434a8
@e32wong
Fix compilation errors with pnpm and vite
153d80c
@e32wong
Fix the zupass where it causes editor to be blank
a760139
@e32wong
Add a utility file for the conversation validations to simplify the code
0a8303e
@e32wong
Add a check for locked conversations and invalid conversation settings
8d806f4
@e32wong
Regenerate openapi files
8b3333f
@e32wong
Fix the error messages
f0b4e30
@e32wong
Fix the placeholder text for locked conversation
6248619
@e32wong
Set the poll ID correctly for conversation addition and edits
13cf9e3
@e32wong
Revise the DTO for the conversation edit API
211cd3a
@e32wong
Fixes to edit page
e266f53
@e32wong
Fix the poll vote reset warning to detect changes properly
2947880
@e32wong
Add backfill script for database for poll ID
b17ecdc
@e32wong
Add the replace poll option into the API and map each vote response to a
015e966 
poll id
@e32wong
Query and error handling fixes
786eeee
@e32wong
Improve the poll component's messaging
6b28936
@e32wong
Add the edited indicator to the conversations
04be599
@e32wong
Fix type error
fd67a90
@e32wong
Fix PrimeVue initialization
b95bb9c
@e32wong
Defer maz ui loading
a8790a3
@e32wong
Lazy load i18n with english first
1f3db8d
@e32wong
Revert "Defer maz ui loading"
ef56cda
@e32wong
Priority authentication call
9351c76
@e32wong
Optimize maz ui and crypto ucan's loading
fb63299
@e32wong
Defer the TipTap editor mounting using a composable
7845ed4
@e32wong
Defer the loading of the QR code library
82b3e42
@e32wong
Remove all manual chunking customizations from Vite
36a77ef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@e32wong