Update entire structure of learn docs #336
Conversation
|
|
||
| ```python exec | ||
|
|
||
| class CallHandlerState(State): |
There was a problem hiding this comment.
Example seems to be broken for me
| @@ -8,7 +8,6 @@ import numpy as np | |||
| import reflex as rx | |||
|
|
|||
| from pcweb.base_state import State | |||
| from pcweb.pages.docs.advanced_guide.custom_vars import custom_vars | |||
| from pcweb.templates.docpage import ( | |||
There was a problem hiding this comment.
Not related to this pr but just noticed the Supported Operations example is not working
There was a problem hiding this comment.
what is broken with it? It seems to work for me.
There was a problem hiding this comment.
For this section we can probably break it up into two files router methods and other methods
docs/pages/page_meta_data.md
Outdated
There was a problem hiding this comment.
This file can be renamed just meta data
pcweb/pcweb.py
Outdated
There was a problem hiding this comment.
yes I just noticed this, will uncomment now
|
Last think think the secitons should be names UI, State. Not frontend and backend |
docs/assets/referencing-assets.md
Outdated
| @@ -0,0 +1,38 @@ | |||
| ```python exec | |||
There was a problem hiding this comment.
I think this file can be called "Upload and Download Files". It may be better for searchability
docs/components/component_props.md
Outdated
| @@ -1,7 +1,7 @@ | |||
| ```python exec | |||
There was a problem hiding this comment.
Can this page be called props.md or is that a name conflict?
docs/pages/meta_data.md
Outdated
| @@ -0,0 +1,46 @@ | |||
| ```python exec | |||
There was a problem hiding this comment.
Should be metadata.py as one word
The page links that are in the sidebar that are not accordions need to be styled a little better