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

Update FAQ content #45

Merged
merged 3 commits into from
Aug 19, 2024
Merged

Update FAQ content #45

merged 3 commits into from
Aug 19, 2024

Conversation

Pabl0cks
Copy link
Member

Updated the FAQ with a proposal of Questions and Answers, feel free to remove/add or tweak! 🙏

Closes #41

Technical note:

Since I added some links and line breaks, I had to use this to make it parse the HTML in order to avoid adding an external library:

<p dangerouslySetInnerHTML={{ __html: item.answer }}></p>

Sounds really bad so I bet it's not a good practice, wondering what would be the best approach in this case.

Maybe it's better to just hardcode all the question and answers in html?

Copy link

vercel bot commented Aug 19, 2024

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

Name Status Preview Comments Updated (UTC)
extensions-hackathon ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 19, 2024 9:52am

Copy link
Contributor

@carletex carletex left a comment

Choose a reason for hiding this comment

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

Great @Pabl0cks

Add some little tweaks. This might need another pass, but I think it's good enough for a merge!

@carletex carletex merged commit be6df7a into main Aug 19, 2024
2 of 3 checks passed
@carletex carletex deleted the faq-content branch August 19, 2024 09:51
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.

FAQ: Add final questions / answers
2 participants