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

Feature/prismic docs #2287

Merged

Conversation

clari182
Copy link
Collaborator

@clari182 clari182 commented Sep 14, 2023

Allows docs to be created via Prismic.
Docs are essentially pages, like about, contact & follow etc

This fetches docs from prismic and available doc mdx files present. It will only create MDX pages with available slugs.
For example if the contact page is in Prismic with slug /contact and an MDX file exists with slug /contact it will only use the prismic one.

@netlify
Copy link

netlify bot commented Sep 14, 2023

Deploy Preview for staging-aiid ready!

Name Link
🔨 Latest commit 8c5c059
🔍 Latest deploy log https://app.netlify.com/sites/staging-aiid/deploys/65367d6ae85b3d0008430e2f
😎 Deploy Preview https://deploy-preview-2287--staging-aiid.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@clari182 clari182 requested a review from pdcp1 October 16, 2023 18:59
Copy link
Collaborator

@pdcp1 pdcp1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good. The only thing that I think is missing is the test of this new feature.

@clari182 clari182 requested a review from pdcp1 October 23, 2023 15:57
@clari182 clari182 requested a review from cesarvarela October 25, 2023 12:21
@clari182
Copy link
Collaborator Author

clari182 commented Nov 7, 2023

@cesarvarela can we merge this?

@cesarvarela cesarvarela merged commit baa42cf into responsible-ai-collaborative:staging Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants