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

Autolab Documentation Improvement #1351

Merged
merged 12 commits into from
May 4, 2021
Merged

Autolab Documentation Improvement #1351

merged 12 commits into from
May 4, 2021

Conversation

xinyis991105
Copy link
Contributor

@xinyis991105 xinyis991105 commented May 4, 2021

Description

This PR improves Autolab's documentation.

Motivation and Context

There's been complaint from within the team that Autolab's documentation is confusing and its aesthetic style is not exactly appreciable. This can discourage some potential new users. Since we are rolling out some new features and the new Docker Compose installation method is increasingly being used, we decided it's a good time to refine the documentation.

How Has This Been Tested?

mkdocs serve and visit local host.
The major changes involve:

  1. An upgraded material theme. Now the doc's color scheme is consistent with Autolab red.
  2. The navigation bar on the right side is removed due to uselessness and distraction.
  3. A top bar is added to aid users in navigating through the sections.
  4. The new top level divisions are Overview, Installation, Getting Started, and Reference. Overview introduces high-level ideas and encourages users to try out demo site's features. Installation includes both the docker compose method and manual methods. Autolab Frontend and Tango backend are both included. Getting Started is user/instructor-directed. Reference is developer directed.
  5. Links and wording have been fixed.
    Screen Shot 2021-05-04 at 9 56 35 PM
    Screen Shot 2021-05-04 at 9 55 47 PM

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation, which is located at Autolab Docs
  • I have updated the documentation accordingly, included in this PR

Other issues / help required

If unsure, feel free to submit first and we'll help you along.

ashleyzhang and others added 12 commits March 14, 2021 15:09
* Request for institution name in the bug template (#1285)

* Modify mkdocs material theme to have tabs and use autolab red

Co-authored-by: Victor Huang <victorhuangwq@gmail.com>
* Setup new page for installation unification

* Split up ubuntu and osx installation pages

* Modify installation docs structure

* Change getting started to link to new installation page

* Separate out commands into its own block

* Change backtick code blocks to identation + colon style

* Move Tango installation instructions to installation tab

* Separate Tango deployment methods in docs

* Improve wording on TLS/SSL, and link to Tango installation section after completing manual Autolab setup

* Add minor change from SSL to TLS/SSL

Co-authored-by: xtrkil <victorhuangwq@gmail.com>
* Setup new page for installation unification

* Split up ubuntu and osx installation pages

* Modify installation docs structure

* Change getting started to link to new installation page

* Separate out commands into its own block

* Change backtick code blocks to identation + colon style

* Move Tango installation instructions to installation tab

* Separate Tango deployment methods in docs

* Improve wording on TLS/SSL, and link to Tango installation section after completing manual Autolab setup

* Add minor change from SSL to TLS/SSL

* Separated overview from getting started; restructured index a little bit.

* Restructure done; about to add in details for overview/introduction

* Added picture for Autolab system.

* Done with the initial draft for overview nad getting-started

Co-authored-by: Fan Pu <zengfanpu@gmail.com>
Co-authored-by: xtrkil <victorhuangwq@gmail.com>
* move initialize_user.sh from docker/ to bin/

* Update ubuntu and osx installation docs on creating initial root user step

* Add init-user script
* Create new reference section

* Add reference overview

* Move vmms setup instructions from vmms api to tango overview

* Updates wording

* Updates wording on reference
* Fixed installation wrong link; did some reorganization on the part of Tango VMMS.

* Replaced VMMS setup instruction.
* Improve Overview.

* Adjust wording based on feedback.
* Make Reference section more catered to developers

* Remove tango.md
* Reworded FAQ to Troubleshooting. Added pointers to troubleshooting sections.

* Fixed wording.
* Ensure all links are valid and outgoing links open in new tab

* Change outgoing link in tango.md to open in new tab
* updating docs with finalized wording

* Documentation FAQ Refinement (#1348)

* Reworded FAQ to Troubleshooting. Added pointers to troubleshooting sections.

* Fixed wording.

* Ensure all links are valid (#1349)

* Ensure all links are valid and outgoing links open in new tab

* Change outgoing link in tango.md to open in new tab

* updating docs with finalized wording

* fixing page linking

Co-authored-by: xinyis991105 <54905587+xinyis991105@users.noreply.github.com>
Co-authored-by: ashleyzhang <ashleyzhang@gmail.com>
* Add smtp settings in production.rb and documentation

* Add instruction updates for development

* Added description of where config files are at

* fix syntax in mkdocs
@xinyis991105 xinyis991105 requested a review from ashleyzhang May 4, 2021 14:00
Copy link
Contributor

@ashleyzhang ashleyzhang left a comment

Choose a reason for hiding this comment

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

LGTM, nice work! 🚢

@xinyis991105 xinyis991105 merged commit 85cd495 into master May 4, 2021
@xinyis991105 xinyis991105 deleted the improve_doc branch May 4, 2021 14:14
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.

5 participants