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

docs: Update docs-app content with installation instructions #12

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

ynotdraw
Copy link
Contributor

@ynotdraw ynotdraw commented Jan 26, 2023

🚀 Description

Updates the docs-app with less TODO/Coming Soon text and a bit more info. In particular, an installation section. Since this is still a private repo, this isn't actually valid yet, as we aren't releasing, but soon™.


🔬 How to Test

CI
CI (Actions) should all be green 🟢 . CI tests linting and tests!

Docs App

Pull down this repo and run the following:

pnpm i
cd docs-app
pnpm start

Go to the docs page and behold all of their glory.


📸 Images/Videos of Functionality

Screenshot 2023-01-26 at 1 33 41 PM

Screenshot 2023-01-26 at 1 33 44 PM

@ynotdraw ynotdraw self-assigned this Jan 26, 2023
@@ -5,7 +5,7 @@
<TopBar>
Copy link
Contributor

Choose a reason for hiding this comment

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

I need to finish templatizing all this 🙃

@ynotdraw ynotdraw marked this pull request as ready for review January 26, 2023 18:40
Copy link
Contributor

@nicolechung nicolechung left a comment

Choose a reason for hiding this comment

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

💯

Base automatically changed from 5-add-the-toucan-button-component to main January 26, 2023 19:14
@ynotdraw ynotdraw merged commit cacad72 into main Jan 30, 2023
@ynotdraw ynotdraw deleted the update-docs-app-content branch January 30, 2023 19:03
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