Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

9 improvement add cms #36

Merged
merged 26 commits into from
Sep 27, 2023
Merged

9 improvement add cms #36

merged 26 commits into from
Sep 27, 2023

Conversation

wongprom
Copy link
Owner

Status Type Env Vars Change Review App Ticket Issue number
Ready Feature No Link Link #9

⚠️ NOTE: Look in to revalidate thing.

Problem

Don't want to have data locally in project.

Solution

  • Add Sanity CMS to store certificates data
  • Have the Studio embedded in route /admin

Before & After Screenshots

BEFORE:
[insert screenshot here]

AFTER:
[insert screenshot here]

Other changes (e.g. bug fixes, UI tweaks, small refactors)

Deploy Notes

Notes regarding deployment of the contained body of work. These should note any
new dependencies, new scripts, etc.

Problem after deployment:

  • route /admin ended up with a black-screen
    • Uninstall node
    • Install node with brew -> nvm. Now running node v18.11.0 and npm v9.6.4

Checklist

General

  • My pull request represents one logical piece of work.
  • My commits are related to the pull request and look clean.
  • I have performed a self-review of my code
  • My code contains no conflicts and is up to date with the latest main branch

wongprom added 25 commits September 12, 2023 13:23
Change noder version.
 after getting later version of node.
Embed Sanity Studio to path /about/certificates/admin. Idea is after login we can add new certificates from Sanity Studio.
Create schema for Certificates. Fetch all certificate with groq. Display title,subtitle with JSX.
Add new content like post, category, auth. Create types for new content. Add visiontool plugin. Refactore some sanity client code.
 Create ClientSideLink client-side component to be used on server-side pages for optimization.
 with slug and display id and title on page.
 to ClientSideLink.tsx. Idea is to render much as possible from server-side, thats why I created this component.
 and remove out-comment code.
 author, category, post
 after pushing this branch. Cant access Studio, black screen. Change useCdn to true froom false.
 from next-sanity 5.4.6 to 5.5.4, sanity 3.16.4 to 3.16.7.
packages in package.json. Remove experimental appdir in next.conf file.
 Create a back arrow for every certificate visited so it is possible to go back to prew page.
@wongprom wongprom linked an issue Sep 27, 2023 that may be closed by this pull request
2 tasks
@vercel
Copy link

vercel bot commented Sep 27, 2023

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

Name Status Preview Comments Updated (UTC)
test-next-13 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 27, 2023 9:56am

@wongprom wongprom deleted the 9-improvement-add-cms branch September 27, 2023 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[IMPROVEMENT] add CMS
1 participant