Skip to content

Added FAQ Page - #1479

Closed
PhoneDroid wants to merge 6 commits into
freshframework:mainfrom
PhoneDroid:FAQ
Closed

Added FAQ Page#1479
PhoneDroid wants to merge 6 commits into
freshframework:mainfrom
PhoneDroid:FAQ

Conversation

@PhoneDroid

@PhoneDroid PhoneDroid commented Jul 18, 2023

Copy link
Copy Markdown

As with any bigger open source project I'm quite
concerned about the count of issues / discussions.

While Fresh is still on the 'lower' side,
I can easily see it getting worse quickly.

To this end I would like to propose - for one -
to extract questions and answers from both
places into a dedicated FAQ and with it,
close the source tickets / discussions.

Steps

  • Add FAQ route
  • Collect questions & answers
  • Process and write them out
  • Close source issues / discussions

To Be Closed

Discussions that can be closed once the FAQ is merged.


image


@PhoneDroid
PhoneDroid marked this pull request as ready for review July 27, 2023 19:42
@PhoneDroid PhoneDroid changed the title [ Draft ][ Documentation ] Added FAQ Page [ 𝗗𝗼𝗰𝘂𝗺𝗲𝗻𝘁𝗮𝘁𝗶𝗼𝗻 ][ 𝓡𝓮𝓿𝓲𝓮𝔀 𝓡𝓮𝓺𝓾𝓲𝓻𝓮𝓭 ] Added FAQ Page Jul 30, 2023
Comment thread deno.json
"exclude": ["www/"]
},
"fmt": {
"exclude": ["docs/faq.md"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why does this file need to be excluded from formatting?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The autoformatting breaks markdown syntax.
I'll happily take the role of janitor for that file ;3

Comment thread docs/faq.md Outdated
Comment thread www/routes/faq.tsx Outdated
Comment thread www/routes/faq.tsx Outdated
Comment thread www/routes/faq.tsx Outdated
Comment thread docs/faq.md

#### Do you offer **i18** support

Not yet, it's on the **[Roadmap]**, however not as a top priority.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This makes it sound i18n wouldn't be possible with Fresh. Sure, we don't have some helpers that make it nicer, but any website can add i18n support without helper functions from the framework.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

How about this instead?
image

Comment thread docs/faq.md
Comment on lines +30 to +36
You can find **Marvin** on the following platforms:

- Discord **@marvin_h** for quick questions

- **[GitHub][Discussions]** for longer form discussions

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this is a good idea. Other users can help other users too. Pointing everyone in my direction means I'll be likely bombarded with support requests that others can help with too.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

How about this?
image

@lucacasonato lucacasonato changed the title [ 𝗗𝗼𝗰𝘂𝗺𝗲𝗻𝘁𝗮𝘁𝗶𝗼𝗻 ][ 𝓡𝓮𝓿𝓲𝓮𝔀 𝓡𝓮𝓺𝓾𝓲𝓻𝓮𝓭 ] Added FAQ Page Added FAQ Page Jul 31, 2023
@bartlomieju

Copy link
Copy Markdown
Contributor

Thanks for putting this together! The idea of an FAQ is nice, but the codebase has been completely rewritten for Fresh 2.x since this PR was opened — the www routes, manifest system, and navigation are all different now. The FAQ content and links would also need updating for the current state of the project.

Closing as stale.

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.

4 participants