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.

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

Update findCapitalProjectsByCityCouncilDistrict to use the transform pipe in param decorators #340

Open
TangoYankee opened this issue Jul 5, 2024 · 0 comments

Comments

@TangoYankee
Copy link
Member

          Oh, interesting. The way we solved this on Horatio's branch was to merge the definitions. Merging the defintions fits the prefixed pipes pattern better. However, it kinda waters down the schema definitions because it relies on "partial".

Using the pipe in each type of parameter might actually be more powerful because it uses the zod schema definition as-is

Originally posted by @TangoYankee in #327 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant