Closed adiati98 closed 2 months ago
Name | Link |
---|---|
Latest commit | 7d644158d56383ded1da899e75d6fb1245809ef1 |
Latest deploy log | https://app.netlify.com/sites/sauced-intro/deploys/661a74b0e1e97100080926ad |
Deploy Preview | https://deploy-preview-138--sauced-intro.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
@BekahHW this works well locally. But the deploy preview gives 404, so I believe this would break the production. I'll try to debug this tomorrow.
I'm not sure how we deploy this course so I might not have access to it. If you can help me to take a look at this, it would be great. TIA! π
[EDIT ADD]: If it's deployed with Netlify, this thread might help? Because now the folder that we want to publish is docs
. Here is another resource on docsify for deploying on Netlify.
@BekahHW I marked this as ready for review just in case you want to pull and play around with it. And if you're ready, you can merge it at anytime (you can't do so in draft mode).
Thanks!
Description
This PR reorganizes intro repository to welcome the new Becoming Maintainer course with below changes:
docs
folder.intro-to-oss
folder indocs
folder to store the current course content._assets
folder.mergeNavbar
to merge navbar to the sidebar for smaller screens.Note
This PR is still a draft waiting for below changes:
What type of PR is this? (check all applicable)
Related Tickets & Documents
Closes #135
Mobile & Desktop Screenshots/Recordings
Steps to QA
Pull this branch.
Run
npm install -g docsify-cli
to install the dependecies.Run
docsify serve docs
to run this project locally.