mm-ninja-turtles / turtle-express

My attempt of making Express.JS a typesafe router with Zod.
3 stars 0 forks source link

[feature]: possible duplicate name conflicts in ctx(context) object of resolver #45

Closed PyaePhyoKyaw61194 closed 2 years ago

PyaePhyoKyaw61194 commented 2 years ago
Screenshot 2022-08-20 at 02 50 38

We should keep reserved names ( params, body , query ... ? ) for property name of custom context to avoid conflicts.

wai-lin commented 2 years ago

ah yes, we should reserve those names. I don't know how to reserve it yet though. 🤣

wai-lin commented 2 years ago

I'll close this issue since v0.2.0 removed the context for the duplicate names and the problem of where the context value came from when we introduce root level context for plugin api.