starred/yaak
1
0
Fork 0
mirror of https://github.com/mountain-loop/yaak.git synced 2026-01-11 20:00:29 +01:00
Code Issues 8 Packages Projects Releases 10 Wiki Activity

[PR #123] [MERGED] Fix performance related to having 100s of requests #58

New Issue
Closed
opened 2025-12-29 07:19:06 +01:00 by adam · 0 comments
adam commented 2025-12-29 07:19:06 +01:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/123
Author: @gschier
Created: 10/8/2024
Status: ✅ Merged
Merged: 10/8/2024
Merged by: @gschier

Base: master ← Head: large-workspace-perf

📝 Commits (5)

📊 Changes

34 files changed (+455 additions, -422 deletions)

View changed files

📝 src-tauri/src/lib.rs (+23 -10)
📝 src-tauri/yaak_models/src/queries.rs (+67 -16)
📝 src-web/components/App.tsx (+3 -1)
📝 src-web/components/AppRouter.tsx (+10 -0)
📝 src-web/components/CookieDialog.tsx (+1 -1)
📝 src-web/components/CookieDropdown.tsx (+1 -1)
📝 src-web/components/DefaultLayout.tsx (+1 -1)
📝 src-web/components/GlobalHooks.tsx (+3 -197)
📝 src-web/components/GrpcConnectionLayout.tsx (+1 -1)
📝 src-web/components/Sidebar.tsx (+42 -19)
📝 src-web/font-size.ts (+1 -1)
📝 src-web/hooks/useActiveCookieJar.ts (+5 -38)
📝 src-web/hooks/useCookieJars.ts (+3 -17)
📝 src-web/hooks/useEnvironments.ts (+3 -14)
📝 src-web/hooks/useFolders.ts (+4 -17)
➕ src-web/hooks/useGenerateThemeCss.ts (+26 -0)
📝 src-web/hooks/useGrpcConnections.ts (+4 -20)
📝 src-web/hooks/useGrpcRequests.ts (+2 -16)
📝 src-web/hooks/useHttpRequestActions.ts (+9 -3)
📝 src-web/hooks/useHttpRequests.ts (+2 -15)

...and 14 more files

📄 Description

  • Port more of the model queries from React Query -> Jotai
  • Move some expensive hooks from SidebarItem to Sidebar
  • Fetch all responses for the workspace, instead of per request
  • Auto-delete responses after there are 20
  • Tweak React Query default settings
  • Move some global hooks above the router, so they don't remount on route change
  • ... probably more stuff...

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mountain-loop/yaak/pull/123 **Author:** [@gschier](https://github.com/gschier) **Created:** 10/8/2024 **Status:** ✅ Merged **Merged:** 10/8/2024 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `master` ← **Head:** `large-workspace-perf` --- ### 📝 Commits (5) - [`1f5c01d`](https://github.com/mountain-loop/yaak/commit/1f5c01db09c50365a476a7cccc036dbb292b4332) A bunch of improvements - [`3dc1a64`](https://github.com/mountain-loop/yaak/commit/3dc1a640a43e46e00228293ad08becd7e975b655) Fix lint error - [`565befe`](https://github.com/mountain-loop/yaak/commit/565befe5ec0bde28adaae9b0cf053dfea72db4b7) Force Vite on dev - [`aae7398`](https://github.com/mountain-loop/yaak/commit/aae73983ef174e4d4f4be7e2b7a0fdded23fb149) Global hooks back into component - [`d9f79bb`](https://github.com/mountain-loop/yaak/commit/d9f79bb04c70371b9d81b716277522eec317809f) Fix pop in ### 📊 Changes **34 files changed** (+455 additions, -422 deletions) <details> <summary>View changed files</summary> 📝 `src-tauri/src/lib.rs` (+23 -10) 📝 `src-tauri/yaak_models/src/queries.rs` (+67 -16) 📝 `src-web/components/App.tsx` (+3 -1) 📝 `src-web/components/AppRouter.tsx` (+10 -0) 📝 `src-web/components/CookieDialog.tsx` (+1 -1) 📝 `src-web/components/CookieDropdown.tsx` (+1 -1) 📝 `src-web/components/DefaultLayout.tsx` (+1 -1) 📝 `src-web/components/GlobalHooks.tsx` (+3 -197) 📝 `src-web/components/GrpcConnectionLayout.tsx` (+1 -1) 📝 `src-web/components/Sidebar.tsx` (+42 -19) 📝 `src-web/font-size.ts` (+1 -1) 📝 `src-web/hooks/useActiveCookieJar.ts` (+5 -38) 📝 `src-web/hooks/useCookieJars.ts` (+3 -17) 📝 `src-web/hooks/useEnvironments.ts` (+3 -14) 📝 `src-web/hooks/useFolders.ts` (+4 -17) ➕ `src-web/hooks/useGenerateThemeCss.ts` (+26 -0) 📝 `src-web/hooks/useGrpcConnections.ts` (+4 -20) 📝 `src-web/hooks/useGrpcRequests.ts` (+2 -16) 📝 `src-web/hooks/useHttpRequestActions.ts` (+9 -3) 📝 `src-web/hooks/useHttpRequests.ts` (+2 -15) _...and 14 more files_ </details> ### 📄 Description - Port more of the model queries from React Query -> Jotai - Move some expensive hooks from SidebarItem to Sidebar - Fetch all responses for the workspace, instead of per request - Auto-delete responses after there are 20 - Tweak React Query default settings - Move some global hooks above the router, so they don't remount on route change - ... probably more stuff... --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 07:19:06 +01:00
adam closed this issue 2025-12-29 07:19:07 +01:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/yaak#58
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?