Simplify import

[PonomareVlad]

As we tested earlier, the "browser" target in the "exports" field is already supported by Vercel Edge and Cloudflare Workers

I suggest shortening the optional import from the web path to get more generic examples

Example source

[github-actions]

🚀 Deployed on https://649b4b789506ac0888cea751--grammy.netlify.app

[rojvv]

Thank you!

[rojvv]

Hmm seems like there is nothing to do here. The imports were already replaced to deno.land/x in main.

[KnorpelSenf]

Perhaps we should do the same for Vercel?

[rojvv]

Convert to deno.land/x import or just grammy?

[KnorpelSenf]

I was thinking of using grammy instead of grammy/web but now that you ask about it, we might actually migrate the guides to the new deno-based service by vercel, right?

[rojvv]

What is the new deno-based service by vercel?

[KnorpelSenf]

https://vercel.com/docs/functions/edge-functions is based on https://deno.com/subhosting

[rojvv]

Oh that makes sense

[rojvv]

I’ll leave that decision to our VP of Serverless, @PonomareVlad.

[PonomareVlad]

Hmm seems like there is nothing to do here. The imports were already replaced to deno.land/x in main.

@roj1512 There is to do

[rojvv]

Thanks!

[rojvv]

What are your thoughts on #834 (comment), @PonomareVlad?

[PonomareVlad]

What are your thoughts on #834 (comment), @PonomareVlad?

There is no any Deno-based service in Vercel 🌚

[PonomareVlad]

Except this community runtime for Serverless Functions

[rojvv]

What about their Edge Runtime? #834 (comment)

[PonomareVlad]

What about their Edge Runtime? #834 (comment)

Earlier I think about extending Vercel guide for Edge Runtime case, but we need to test which grammY plugins still incompatible with it and notify about this before announcing compatibility 🌚

For example

[rojvv]

So you mean the grammy import is better for now, right?

[PonomareVlad]

So you mean the grammy import is better for now, right?

Is identical to grammy/web for supported runtimes that understand browser exports

[rojvv]

So it’ll be fine for Vercel, right?

[PonomareVlad]

So it’ll be fine for Vercel, right?

Including Vercel, right

[rojvv]

Can you create a PQ for that?

[PonomareVlad]

Yes, but it depends on:

What about their Edge Runtime? #834 (comment)

Earlier I think about extending Vercel guide for Edge Runtime case, but we need to test which grammY plugins still incompatible with it and notify about this before announcing compatibility 🌚

For example

[rojvv]

I meant for the Node.js example, not their edge runtime.

[PonomareVlad]

I meant for the Node.js example, not their edge runtime.

In docs or in repo with examples ?