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

Implement foundation page as markdown, with MarkdownTemplate.astro #76

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

rfay
Copy link
Member

@rfay rfay commented Aug 15, 2023

The Issue

Watching https://craftquest.io/courses/astro-quick-start-guide/50358 I was inspired to try to make top-level markdown work on ddev.com

How This PR Solves The Issue

Uses the basic approaches there along with @mattstein suggestions, especially the Tailwind "prose" class. It works OK at this point but is not ready for prime time.

* I couldn't get it to build if I added in the <SEO> component

  • It's apparent to me that I don't know what I'm doing having MarkdownLayout.astro import Layout.astro

Manual Testing Instructions

https://20230811-markdown-pages.ddev-com-front-end.pages.dev/foundation/

Thanks for your help so far @mattstein - I'd be happy to book a session with you to look at this or also happy to receive async comments.

@rfay rfay requested a review from mattstein August 15, 2023 00:03
@cloudflare-pages
Copy link

cloudflare-pages bot commented Aug 15, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: fdeac07
Status: ✅  Deploy successful!
Preview URL: https://f4d2d55c.ddev-com-front-end.pages.dev
Branch Preview URL: https://20230811-markdown-pages.ddev-com-front-end.pages.dev

View logs

@rfay rfay marked this pull request as ready for review August 24, 2023 00:29
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.

None yet

1 participant