WIP: feat: add zod schemas #463

Closed
opened 2025-11-01 20:56:32 -05:00 by GiteaMirror · 0 comments
Owner

Originally created by @vikunja-bot on GitHub (Apr 1, 2025).

Original issue by @kolaente on 2024-02-08T13:16:20.000Z

Cherry-picked from https://kolaente.dev/vikunja/frontend/pulls/2225

Original issue on Gitea


dpschen commented on 2025-01-14T14:47:16.000Z:

@konrad Do you remember what exactly you did when rebasing?

Asking mostly because in this PR there are a lot more changes than in the original PR. The original PR added the zod schemas (aka new files) that were not used yet and added two dependencies (zod and validator.js for some validation). See also the commits.

The idea was to iteratively switch to zod. If you want I can "rebase" the original PR, but I would need to know what else changed here.

Tbh I would actually redo the changes since that seems simpler:

  1. add the new schema files to the modelSchema folder
  2. add the dependencies
  3. adjust model changes since the PR

@kolaente commented on 2025-01-14T16:52:49.000Z:

IIRC I tried to apply the changes that I did when merging the frontend and api repos.


dpschen commented on 2025-01-16T11:51:44.000Z:

IIRC I tried to apply the changes that I did when merging the frontend and api repos.
I realized that there were just some additional commits that added, that were already part of the main branch. When I rebased this branch they vanished.

Originally created by @vikunja-bot on GitHub (Apr 1, 2025). _Original issue by @kolaente on 2024-02-08T13:16:20.000Z_ Cherry-picked from https://kolaente.dev/vikunja/frontend/pulls/2225 [Original issue on Gitea](https://kolaente.dev/vikunja/vikunja/issues/2093) --- _**dpschen** commented on 2025-01-14T14:47:16.000Z_: @konrad Do you remember what exactly you did when rebasing? Asking mostly because in this PR there are a lot more changes than in the original PR. The original PR added the zod schemas (aka new files) that were not used yet and added two dependencies (zod and validator.js for some validation). See also [the commits](https://kolaente.dev/vikunja/frontend/pulls/2225/commits). The idea was to iteratively switch to zod. If you want I can "rebase" the original PR, but I would need to know what else changed here. Tbh I would actually redo the changes since that seems simpler: 1. add the new schema files to the `modelSchema` folder 2. add the dependencies 3. adjust model changes since the PR --- _**@kolaente** commented on 2025-01-14T16:52:49.000Z_: IIRC I tried to apply the changes that I did when merging the frontend and api repos. --- _**dpschen** commented on 2025-01-16T11:51:44.000Z_: > IIRC I tried to apply the changes that I did when merging the frontend and api repos. I realized that there were just some additional commits that added, that were already part of the main branch. When I rebased this branch they vanished.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#463