Static preview - Githubissues

pwizla commented 1 week ago

This PR adds docs for Static Preview (it targets the beta version, so no mention of feature flags):

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
documentation	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 29, 2024 1:17pm

remidej commented 1 week ago

This is really good!

2 things about the next.js guide part:

with the code you suggest they'll run into the issue where they will never exit Next.js draft mode after opening it. This is the bug you encountered yesterday where you see draft content even in the published tab. You can see how I fixed in in the launchpad. Basically you need to always go through the next.js preview route, so that you can force the draft mode cookie removal when the status is published.
since your guide is pretty in depth already, I would add the last missing piece, which is to check if draft mode is enabled in the places that are querying Strapi data, so they can know whether they should add the status=draft param. See this part of the launchpad code for an example.

yanniskadiri commented 4 days ago

I just checked the user guide part, it looks good to me. This is great work, thanks @pwizla!

pwizla commented 5 hours ago

@remidej I did my best to improve the implementation guide based on your feedback 😊 What do you think?

strapi / documentation