[PR #193] [MERGED] Generalized frontend model store #118

Closed
opened 2025-12-29 07:19:25 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/193
Author: @gschier
Created: 3/26/2025
Status: Merged
Merged: 3/31/2025
Merged by: @gschier

Base: masterHead: frontend-model-store


📝 Commits (10+)

  • 02660ae A great start to generalized store
  • 4a0a7e3 Migrate the rest of the models
  • bbf68a1 Fewer re-renders
  • 28e581f Remove some upsert/create hooks
  • 8e4e375 Port delete operations too
  • 51bc503 Model duplication
  • 5b4e96c Remove unused Tauri commands
  • 2d81569 Fix key values and websocket request cloning
  • 7259348 Fix key value watching
  • bf5078e Websocket and GRPC event handling

📊 Changes

201 files changed (+2177 additions, -2857 deletions)

View changed files

📝 package-lock.json (+6 -4)
📝 package.json (+3 -0)
📝 src-tauri/gen/schemas/acl-manifests.json (+1 -1)
📝 src-tauri/gen/schemas/desktop-schema.json (+50 -0)
📝 src-tauri/gen/schemas/macOS-schema.json (+50 -0)
src-tauri/migrations/20250326193143_key-value-id.sql (+43 -0)
📝 src-tauri/src/http_request.rs (+2 -3)
📝 src-tauri/src/lib.rs (+13 -466)
📝 src-tauri/src/updates.rs (+1 -2)
📝 src-tauri/yaak-license/src/license.rs (+1 -1)
📝 src-tauri/yaak-models/bindings/gen_models.ts (+1 -1)
📝 src-tauri/yaak-models/build.rs (+9 -1)
src-tauri/yaak-models/guest-js/atoms.ts (+80 -0)
src-tauri/yaak-models/guest-js/index.ts (+10 -0)
src-tauri/yaak-models/guest-js/store.ts (+199 -0)
src-tauri/yaak-models/guest-js/types.ts (+8 -0)
src-tauri/yaak-models/guest-js/util.ts (+23 -0)
src-tauri/yaak-models/index.ts (+0 -8)
📝 src-tauri/yaak-models/package.json (+1 -1)
src-tauri/yaak-models/permissions/autogenerated/commands/duplicate.toml (+13 -0)

...and 80 more files

📄 Description

This PR removes all the bespoke frontend model handling, replacing it with some generalized helpers for upsert/list/get/delete/duplicate.


🔄 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/193 **Author:** [@gschier](https://github.com/gschier) **Created:** 3/26/2025 **Status:** ✅ Merged **Merged:** 3/31/2025 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `master` ← **Head:** `frontend-model-store` --- ### 📝 Commits (10+) - [`02660ae`](https://github.com/mountain-loop/yaak/commit/02660ae2c4a1befe5e55289f3daaab41397248d3) A great start to generalized store - [`4a0a7e3`](https://github.com/mountain-loop/yaak/commit/4a0a7e3b4fd22d1fd76619fe43b2260704a81fb3) Migrate the rest of the models - [`bbf68a1`](https://github.com/mountain-loop/yaak/commit/bbf68a1e31931443759d48d583fd67c121ea98c4) Fewer re-renders - [`28e581f`](https://github.com/mountain-loop/yaak/commit/28e581f75896720157395fd3adc713e4dd69a9cf) Remove some upsert/create hooks - [`8e4e375`](https://github.com/mountain-loop/yaak/commit/8e4e375d57306628d30560547313b50e4a9bddfa) Port delete operations too - [`51bc503`](https://github.com/mountain-loop/yaak/commit/51bc50397a2e839d2982d6620a3ba259104a5545) Model duplication - [`5b4e96c`](https://github.com/mountain-loop/yaak/commit/5b4e96cf91531b875387d32ac9210a764e648c71) Remove unused Tauri commands - [`2d81569`](https://github.com/mountain-loop/yaak/commit/2d81569463f660785a5700b028d7eb177cd07398) Fix key values and websocket request cloning - [`7259348`](https://github.com/mountain-loop/yaak/commit/7259348e854ce8c07e669e668639fa3d50f6c528) Fix key value watching - [`bf5078e`](https://github.com/mountain-loop/yaak/commit/bf5078e55fd2871fb32f064890f23c06fdbd1e1a) Websocket and GRPC event handling ### 📊 Changes **201 files changed** (+2177 additions, -2857 deletions) <details> <summary>View changed files</summary> 📝 `package-lock.json` (+6 -4) 📝 `package.json` (+3 -0) 📝 `src-tauri/gen/schemas/acl-manifests.json` (+1 -1) 📝 `src-tauri/gen/schemas/desktop-schema.json` (+50 -0) 📝 `src-tauri/gen/schemas/macOS-schema.json` (+50 -0) ➕ `src-tauri/migrations/20250326193143_key-value-id.sql` (+43 -0) 📝 `src-tauri/src/http_request.rs` (+2 -3) 📝 `src-tauri/src/lib.rs` (+13 -466) 📝 `src-tauri/src/updates.rs` (+1 -2) 📝 `src-tauri/yaak-license/src/license.rs` (+1 -1) 📝 `src-tauri/yaak-models/bindings/gen_models.ts` (+1 -1) 📝 `src-tauri/yaak-models/build.rs` (+9 -1) ➕ `src-tauri/yaak-models/guest-js/atoms.ts` (+80 -0) ➕ `src-tauri/yaak-models/guest-js/index.ts` (+10 -0) ➕ `src-tauri/yaak-models/guest-js/store.ts` (+199 -0) ➕ `src-tauri/yaak-models/guest-js/types.ts` (+8 -0) ➕ `src-tauri/yaak-models/guest-js/util.ts` (+23 -0) ➖ `src-tauri/yaak-models/index.ts` (+0 -8) 📝 `src-tauri/yaak-models/package.json` (+1 -1) ➕ `src-tauri/yaak-models/permissions/autogenerated/commands/duplicate.toml` (+13 -0) _...and 80 more files_ </details> ### 📄 Description This PR removes all the bespoke frontend model handling, replacing it with some generalized helpers for upsert/list/get/delete/duplicate. --- <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:25 +01:00
adam closed this issue 2025-12-29 07:19:25 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/yaak#118