update Tina to latest version #61
Conversation
|
@logan-anderson is attempting to deploy a commit to the Blazity Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
.tina/config.ts
Outdated
| publicFolder: 'public', | ||
| }, | ||
| schema, | ||
| branch, |
There was a problem hiding this comment.
is it still possible to use tina locally? I don't want to push user to use tina cloud
There was a problem hiding this comment.
@bmstefanski What is the error you are seeing?
There was a problem hiding this comment.
Everything should work locally
There was a problem hiding this comment.
Looking into this more, are you talking about building the App without needing Tina Cloud integration?
Right now we promote using Tina Cloud on the server side (even on CI). I am guessing that vercel error is something like this
Error: Client not configured properly. Missing branch, clientId, token. Please visit https://tina.io/docs/tina-cloud/connecting-site/ for more information
This is because to build your app you need to query tina cloud.
If you want to build the app without Tina Cloud the build command would have to look like this.
build: "tinacms dev -c 'next build' "
However this would not build admin interface for production.
I will make this the default build script (so the site will build without TinaCloud) and update the README to add docs of how to edit the script so the site can be used with TinaCloud and you can edit in production.
There was a problem hiding this comment.
I got this error when deploying to vercel or trying to build in local
Error occurred prerendering page "/admin/[[...tina]]". Read more: https://nextjs.org/docs/messages/prerender-error
Error: Unable to find `TinaProvider`; did you forget to add the TinaCMS container to your app root? See our setup docs: https://tina.io/docs/introduction/tina-init/#adding-tina
There was a problem hiding this comment.
@minhdna I just updated again to the latest version of Tina and updated to the latest master. The /admin/[...tina] file was delete are you sure you are deploying from my fork using the correct branch?
It would be really nice to get this merged as we are getting folks asking about tina being broken on this site.
Thanks again for looking at this 😄
|
Hello folks, amazing work! Thanks for the great work! |
|
Hello again @logan-anderson and @bmstefanski! Thank you for this fix! i'd really love for this to be merged so I can use this beautiful template, thank you! |
Hello Team,
Logan here from from the Tina team. Was just looking over at the starter and it seems that you are on an older version of Tina.
Here is a PR that updates it to the latest version and fixes all of the breaking changes. Let me know if you have any questions.