search

oliver-oloughlin / kvdex

A high-level abstraction layer for Deno KV with zero third-party dependencies by default 🦕🗝️
https://jsr.io/@olli/kvdex
MIT License
188 stars 6 forks source link

Relative import path "@olli/kvdex/ext/zod" not prefixed with.... on deno deploy #194

Closed waptik closed 6 months ago

waptik commented 6 months ago

Is this a deno deploy bug because it works perfectly locally? If it is, i will open an issue about in their feedback repo and ping the discord channel. I even committed deno.lock file to git and the issue still persisted. I'll give jsr:@olli/kvdex/ext/zod a try after this and see how it goes

image image image
waptik commented 6 months ago

So using jsr:@olli/kvdex/ext/zod works instead of @olli/kvdex/ext/zod which is weird. I think it's a deno deploy bug. I'm reporting the issue there as well

oliver-oloughlin commented 6 months ago

Thats very strange, I think it might be a bug on Deploy when importing package modules then. But I will have to check as well.

oliver-oloughlin commented 6 months ago

I think it could be that the import map doesn't do what you expect it to. You would maybe have to add a separate entry for "@olli/kvdex/ext/zod": "jsr:@olli/kvdex@^0.35.2/ext/zod"

waptik commented 6 months ago

I think it could be that the import map doesn't do what you expect it to. You would maybe have to add a separate entry for "@olli/kvdex/ext/zod": "jsr:@olli/kvdex@^0.35.2/ext/zod"

Done. Thanks I thought this update fixed it! Probably didn't take into account jsr modules at that time