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

Refactor faq #60

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

versecafe
Copy link
Contributor

This moves the FAQ from a huge stack of HTML/Astro syntax over to easy to edit MDX files with compile time checks making it far easier to add, remove, or adjust them as needed, a new FAQ is as simple as

src/content/faq/[name]

---
question: Why use Astro
---

We moved over from HTML/CSS to Astro to make site maintenance easier on the
team and make adding new content for updates, posts, newsletters, new sponsors,
and more easier.

Note that by using MDX instead of MD it gives support for custom formatting if ever needed, some places had <br /> tags injected to make spacing look nicer, or may in the future need a <img> tag or custom tailwind for a gradient word.

@versecafe versecafe changed the title Chore: Refactor faq Refactor faq Jul 20, 2024
Copy link
Member

@ADKaster ADKaster left a comment

Choose a reason for hiding this comment

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

The move away from a single file for both donate and faq is nice, but the separate files for each faq entry means that the faq entries are arbitrarily shuffled. We organized them in a specific order for a reason, intended to be read in that order.

So either slapping them all into one mdx file, or some way to describe an order for them needs to happen before this is ready to go.

src/components/landing/faq.astro Show resolved Hide resolved
@gmta
Copy link
Contributor

gmta commented Nov 8, 2024

This now has some conflicts.

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