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.

Sign up for GitHub

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

Jump to bottom

remove tutorial organization by day, add summary table #152

Merged
merged 3 commits into from
Sep 1, 2021
Merged

remove tutorial organization by day, add summary table #152

merged 3 commits into from
Sep 1, 2021

Conversation

scottyhq
Copy link
Member

Fixes #149

@github-actions
Copy link

Binder 👈 Launch a binder notebook on this branch

@scottyhq scottyhq added the preview create netlify website preview label Aug 27, 2021
@github-actions
Copy link

github-actions bot commented Aug 27, 2021 •
edited
Loading

🚀 Deployed on https://deploy-preview-152--snowex-hackweek.netlify.app

@scottyhq scottyhq requested a review from dshean August 28, 2021 01:18
@jomey
Copy link
Member

jomey commented Aug 28, 2021

I like the revamp!

Looks like the LIDAR is missing a TOC on the index page:
https://deploy-preview-152--snowex-hackweek.netlify.app/tutorials/lidar/index.html

I can see that people will get confused and not find the two tutorials.

@scottyhq
Copy link
Member Author

scottyhq commented Sep 1, 2021

@jomey i think this is good to go now, could you please approve / merge

@jomey jomey merged commit cb8372b into snowex-hackweek:main Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jomey jomey jomey approved these changes

@dshean dshean Awaiting requested review from dshean

Assignees
No one assigned
Labels
preview create netlify website preview
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make tutorials more prominent
2 participants
@scottyhq @jomey