WIP: feature/zod-schemas #2225

Open
dpschen wants to merge 1 commits from dpschen/frontend:feature/zod-schemas into main
Collaborator

This builds upon the abstract service ts branch.
First I separated the model types from the models.

Then I experiment a bit with zod.
It seems like using such a tool could help us a lot having good typing everywhere. I think most models can completely be replaced by zod schemas. The few methods that the models have could be helpers that you import on demand.

EDIT: Diff is better visible here: https://kolaente.dev/dpschen/frontend/compare/feature/convert-abstract-service-to-ts...feature/zod-schemas

This builds upon the abstract service ts branch. First I separated the model types from the models. Then I experiment a bit with [zod](https://github.com/colinhacks/zod). It seems like using such a tool could help us a lot having good typing everywhere. I think most models can completely be replaced by zod schemas. The few methods that the models have could be helpers that you import on demand. EDIT: Diff is better visible here: https://kolaente.dev/dpschen/frontend/compare/feature/convert-abstract-service-to-ts...feature/zod-schemas
dpschen added 21 commits 4 months ago
Poster
Collaborator

I have no clue if yup would make more sense to use. Since I don't have experience with these kind of libs I just picked one.

I have no clue if [yup](https://github.com/jquense/yup) would make more sense to use. Since I don't have experience with these kind of libs I just picked one.
dpschen force-pushed feature/zod-schemas from 48a83a4fd8 to 8cc08582be 4 months ago
dpschen force-pushed feature/zod-schemas from 8cc08582be to 32b3078ee1 4 months ago
dpschen force-pushed feature/zod-schemas from 32b3078ee1 to b3ae11e0c3 3 months ago
dpschen force-pushed feature/zod-schemas from b3ae11e0c3 to 8705ebbe45 3 months ago
dpschen force-pushed feature/zod-schemas from 8705ebbe45 to 4fcd63c8b6 3 months ago
Collaborator

Hi dpschen!

Thank you for creating a PR!

I've deployed the changes of this PR on a preview environment under this URL: https://2225-feature-zod-schemas--vikunja-frontend-preview.netlify.app

You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/.

Have a nice day!

Beep boop, I'm a bot.

Hi dpschen! Thank you for creating a PR! I've deployed the changes of this PR on a preview environment under this URL: https://2225-feature-zod-schemas--vikunja-frontend-preview.netlify.app You can use this url to view the changes live and test them out. You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/. Have a nice day! > Beep boop, I'm a bot.
dpschen force-pushed feature/zod-schemas from 4fcd63c8b6 to 5198e6b06e 3 months ago
dpschen force-pushed feature/zod-schemas from 5198e6b06e to 8a5017b2d6 3 months ago
dpschen added the
help wanted
kind/feature
labels 2 months ago
dpschen self-assigned this 2 months ago
All checks were successful
continuous-integration/drone/pr Build is passing
Required
Details
This pull request has changes conflicting with the target branch.
package.json
src/modelSchema/common/repeats.ts
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/frontend#2225
Loading…
There is no content yet.