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

Introduce cookbook documentation #1826

Open
wants to merge 2 commits into
base: 2.x
Choose a base branch
from

Conversation

WebMamba
Copy link
Collaborator

@WebMamba WebMamba commented May 4, 2024

Q A
Bug fix? no
New feature? yes
Issues
License MIT

This PR introduces a new section for the UX website: cookbook. This section aims to give more concrete documentation, show examples of common things implemented with UX components, and go deeper into some concepts that we do in the documentation. The goal is not to replace the documentation but to illustrate
the documentation with concrete examples.

Capture d’écran 2024-05-04 à 17 14 24 Capture d’écran 2024-05-04 à 17 15 38

How it's implemented?

Simply by creating a .md file in the recipes directory at the route of the project, then everything is generated automatically. There are cookbook pages that show all the recipes available, then you can click on the recipe to read it.

Who can add recipes to the cookbook?

Anyone! If you thing you implemented something a bit complex but common, just show you did it!

Thank you!
Tell me what you think 😁

@carsonbot carsonbot added Feature New Feature Status: Needs Review Needs to be reviewed labels May 4, 2024
@WebMamba WebMamba force-pushed the introduce_cookbook_documentation branch 5 times, most recently from 9cc6c95 to cfeb25a Compare May 4, 2024 16:19
ux.symfony.com/src/Service/RecipeRepository.php Outdated Show resolved Hide resolved
ux.symfony.com/src/Service/RecipeParser.php Outdated Show resolved Hide resolved
ux.symfony.com/templates/components/RecipeList.html.twig Outdated Show resolved Hide resolved
ux.symfony.com/templates/cookbook.html.twig Outdated Show resolved Hide resolved
ux.symfony.com/templates/recipe.html.twig Show resolved Hide resolved
ux.symfony.com/src/Service/RecipeParser.php Outdated Show resolved Hide resolved
@smnandre
Copy link
Collaborator

smnandre commented May 5, 2024

I'll review tomorrow or tuesday in depth.. but the overall mood: i love it !

Copy link
Collaborator

Choose a reason for hiding this comment

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

As we will add new images / assets more and more, we'll need some tools to normalize image compression + thumbnails.. at least, in the first weeks/monthes, some "process" to manage files before pushing them.

I'll manually work on this one, but when we have time i think some cli tool / command, or maybe a github action, would be a nice addition ! WDYT ?

The other question i'm not sure about is: how many images do we accept ? As we discussed already about the LiveMemory, we do not want the repository to be 1To in a year. Because every media will add to the repository global size, having an impact on the DX for anyone wanting to open a PR.

BUT i'm really the first one to say it's best/easier for us to have images in the repo, so maybe set some "conventions" about it ? I don't know...

Other point of attention concerns the image rights: any file/media we add there must be releasable under the MIT Licence of this repository: how do we check/enforce this ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

component_architecture

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes completely true! I think we need to find a way to manage images for the website. This out of scope for this PR but this really something we should think about!

@ludovik-dev
Copy link

It's a bit related to the PR #1347?

@smnandre
Copy link
Collaborator

smnandre commented May 7, 2024

It's a bit related to the PR #1347?

A bit .. but the "recipe" term refer to a "cookbook recipe" more than a "component recipe" :)

Copy link
Member

@kbond kbond left a comment

Choose a reason for hiding this comment

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

I love this @WebMamba!

I'm thinking, accross the board, we should use the term "cookbook" to not be confused with the future "recipes" idea.

For the markdown, take a look at the front matter extension. I think this could be leveraged here.

@smnandre
Copy link
Collaborator

smnandre commented May 8, 2024

How do you want to proceed @WebMamba here ? Do we review both content / feature here ? Or you prefer just the "code" part for now maybe ?

@WebMamba WebMamba force-pushed the introduce_cookbook_documentation branch from f9f819e to 74fd42d Compare May 12, 2024 16:08
@WebMamba WebMamba force-pushed the introduce_cookbook_documentation branch from 74fd42d to 4552ab1 Compare May 12, 2024 16:11
@WebMamba
Copy link
Collaborator Author

@smnandre lets focus on the code at first, and then on the content. Even if we do that in two times, I can keep the content of the recipe on this PR, I think it's better if we launch the cookbook with a first article.

@kbond thanks, I use the front matter extension now, this is pretty cool!

@codisart thanks for your review I applied your fix!

@smnandre
Copy link
Collaborator

As soon as i find 15mn i'll review it i swear 😮‍💨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New Feature Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants