Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
docs/developer/extending/apps/developing-apps/app-sdk/api-handlers.mdx
Outdated
Show resolved
Hide resolved
docs/developer/extending/apps/developing-apps/app-sdk/api-handlers.mdx
Outdated
Show resolved
Hide resolved
docs/developer/extending/apps/developing-apps/app-sdk/api-handlers.mdx
Outdated
Show resolved
Hide resolved
|
|
||
| export default createManifestHandler({ | ||
| manifestFactory({ request, appBaseUrl }) { | ||
| manifestFactory({ request, appBaseUrl, schemaVersion }) { |
There was a problem hiding this comment.
Question: do we need here schemaVersion? I mean is something we recommend to do with it?
There was a problem hiding this comment.
I wanted to show all available options here (and there are not much). I dont have strong opinion
There was a problem hiding this comment.
Suggestion: maybe it will be better to have next-api-handlers & aws-api-handlers files?
There was a problem hiding this comment.
I added examples. I think it's fine to add dedicated pages too. But I suggest to do this in next PRs.
I wanted this PR to contain everything required with v1:
- Data to be up to date / accurate
- Migration guide
Maybe in next PRs we can extend docs with additional things?
witoszekdev
left a comment
witoszekdev
left a comment
There was a problem hiding this comment.
lgtm, we can also update our app tutorial and add links to templates for each platform (Cloudflare, Deno, etc.) I already made a PR with these changes: https://github.com/saleor/saleor-docs/pull/1437/files
docs/developer/extending/apps/developing-apps/app-sdk/migration-0.x-to-1.x.mdx
Outdated
Show resolved
Hide resolved
docs/developer/extending/apps/developing-apps/app-sdk/migration-0.x-to-1.x.mdx
Outdated
Show resolved
Hide resolved
docs/developer/extending/apps/developing-apps/app-sdk/migration-0.x-to-1.x.mdx
Outdated
Show resolved
Hide resolved
docs/developer/extending/apps/developing-apps/app-sdk/overview.mdx
Outdated
Show resolved
Hide resolved
|
|
||
| app-sdk provides support for multiple platforms. To import specific platform, ensure that you are using the correct import path, for example: | ||
| - For AWS Lambda `import { SaleorAsyncWebhook } from "@saleor/app-sdk/handlers/aws-lambda";` | ||
| - For NextJS App Router `import { SaleorAsyncWebhook } from "@saleor/app-sdk/handlers/next-app-router";` |
There was a problem hiding this comment.
nitpick: let's add link to page where we list all available platforms
docs/developer/extending/apps/developing-apps/app-sdk/api-handlers.mdx
Outdated
Show resolved
Hide resolved
3 participants
saleor/app-sdk#394