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

update website asset links #750

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jxjshaw
Copy link
Contributor

@jxjshaw jxjshaw commented Jul 15, 2022

Summary

I was able to recover the missing files thanks to internet archive. https://web.archive.org/web/20211101114812/https:/webviz.io
https://web.archive.org/web/20210625040821/https://webviz.io/app/?demo

Since the issue has been open for a week without a response, i figured it's better to have the files up somewhere than not. So i'm hosting them on my personal gh page.

It's definitely more ideal to host this somewhere out in the open that anyone can maintain - I would very much welcome someone either Cruise or the community to do that. But in absence of that, figured having them living somewhere is better than nowhere

  • [x ] Fix demo website assets (now hosted on github pages)

  • Change location of demo bag <- to do, i'm currently waiting on the bag download from internet archive...trying to avoid torrenting the giant file from udacity again

Note on the diff: looks like some kind of linter is burying the changes, but all i did was replace these files paths
https://open-source-webviz-ui.s3-us-west-2.amazonaws.com/landing/NAME.gif
with these kinds of file paths:
https://media.githubusercontent.com/media/jxjshaw/jxjshaw.github.io/master/assets/images/work/cruise/webviz-site-backup/gif/NAME.gif

Test plan

Verified the landing page locally

Versioning impact

Does this change necessitate a major, minor, or patch semver update for the affected package?
No

@jxjshaw jxjshaw marked this pull request as draft July 15, 2022 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant