Skip to content
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

chores(svelte): migration to svelte 5 #1685

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

chores(svelte): migration to svelte 5 #1685

wants to merge 19 commits into from
+3,244 −1,782

Conversation

nsarrazin
Copy link
Collaborator

@nsarrazin nsarrazin commented Feb 6, 2025 •
edited
Loading

tested the following flows, everything seems to work

  • new conversation
  • follow up message in conversation
  • editing user message
  • retrying new response
  • deleting branch in conv
  • renaming conv
  • deleting conv
  • creating assistant
  • editing assistant
  • creating tool
  • using tool
  • browsing assistants
  • changing user settings & system prompts

@nsarrazin nsarrazin added front This issue is related to the front-end of the app. back This issue is related to the Svelte backend or the DB dependencies Pull requests that update a dependency file labels Feb 6, 2025
@nsarrazin
chores(svelte5): remove preventDefault & stopPropagation legacy handlers
4357a6c
@nsarrazin
chores(deps): version bumps
7c476c4
@nsarrazin
fix: closing modal works properly
0a2f4a4
@nsarrazin
fix: messagte loading reactivity
1dc8b63
@nsarrazin
Merge branch 'main' into chores/svelte5_migration
3b975b0
@nsarrazin
fix: loading indicator
3ae72e4
@nsarrazin
fix: conversation list reactivity
ffd96ca
@nsarrazin
fix: bug on tool logo rendering
5c394f8
@nsarrazin
fix: infinite loading circular dep
f1d2c8b
@nsarrazin
feat: hide new chat button on new chats
8f0451e
@nsarrazin
fix: move apply directive to main.css
0b982e7
@nsarrazin
Merge branch 'main' into chores/svelte5_migration
bf45819
@nsarrazin
fix: remove legacy syntax in TokensCounter.svelte
13e8d5c
@nsarrazin
fix: replace legacy run with Svelte 5 $effect in NavMenu
1819341
@nsarrazin
fix: replace legacy run with Svelte 5 $effect in ScrollToPreviousBtn
70947d6
@nsarrazin
fix: migrate settings/[modelId] page to Svelte 5 syntax
20de885
@nsarrazin
fix: replace legacy run with Svelte 5 $effect in ToolUpdate compo…
b3578c0 
…nent
@nsarrazin
fix: migrate ToolEdit component to Svelte 5 syntax
4d5c946
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
back This issue is related to the Svelte backend or the DB dependencies Pull requests that update a dependency file front This issue is related to the front-end of the app.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nsarrazin