chore(examples): update with-apivideo

[MarDi66]

Documentation / Examples

Hello Next.js 👋🏼

This PR aims to update the previous with-apivideo-uploader example by:

• Adding others api.video's features
• Changing the example name to with-apivideo

What's new

In addition to the uploader page, we have added:

• A videos list page to help the users display their videos to anyone easily
• A video player page to watch any video from the user's videos list

What do we use

In this example, we use several Next.js features:

• Routes and dynamic routes
• API and dynamic API routes
• SWR

And api.video tools:

• NodeJS client
• React player (instead of our Player SDK)
• TypeScript uploader

Thank's in advance for your review 🙏🏼

[MarDi66]

I see an error in the build test process:
ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up-to-date with packages/next-mdx/package.json

I've already faced this error trying to deploy to Vercel, and the only solution I found was to change the default install command to npm install.

FYI, pnpm i works fine locally.

Any other solution that I can apply (if needed)?

[balazsorban44]

Hi @MarDi66 it looks like contributors are not allowed to edit your branch, so I cannot update or even merge this PR. I wanted to bring in the latest canary to see if that fixed pnpm issues. FWIW, examples should not modify the lock file, so please revert that change. Could you make sure that contributors are allowed to make changes? 🙏 Without that, this PR cannot land.

Further, keep in mind if we change or rename the directory, already existing links on the internet might break, but given the example is a few months old only, I think it is reasonable to rename without keeping the old one with a README that points to the new location. 👍

For simplicity's sake, we like to focus examples on the core feature/library they are presenting. Is SWR a vital part of the example? If not, maybe we should strip it, what do you think?

[MarDi66]

Hey @balazsorban44 ✌🏼

I've removed the lock file changes and SWR from this PR.
I also invited you to our forked Next.js repo, tell me if he lets you manage this PR the way you want.

Cheers!

[balazsorban44]

Thanks!