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

Menu in 'Transferring Data Between your Local Machine and the Cloud' not working #186

Closed
aschuerch opened this issue Apr 2, 2019 · 5 comments
Closed

Menu in 'Transferring Data Between your Local Machine and the Cloud' not working #186

aschuerch opened this issue Apr 2, 2019 · 5 comments
Assignees
@ErinBecker
Labels
help wanted Looking for Contributors

Comments

@aschuerch
Copy link
Contributor

In 05-writing-scripts. The choice menu for "AWS_Windows" and "AWS_Unix" seems to not do something at all (tested in Firefox). I suspect it is a missing javascript paragraph as seen in https://github.com/datacarpentry/cloud-genomics/blob/gh-pages/_episodes/02-logging-onto-cloud.md
@aschuerch aschuerch added the help wanted Looking for Contributors label Apr 2, 2019
ErinBecker added a commit that referenced this issue Apr 30, 2019
@ErinBecker
Enable menu to select platform
330f5a1 
Partial fix for #186

I've added the code to get the menu to work (I think), but as far as I can tell, the content for the section "Uploading Data to your Virtual Machine" for Unix is missing. This PR should **NOT** be merged without adding that content first. I haven't used scp in years. Anyone who can help add this content?
@ErinBecker ErinBecker mentioned this issue Apr 30, 2019
Closed
@ErinBecker ErinBecker added the status:in progress Contributor working on issue label Apr 30, 2019
@ErinBecker
Copy link
Contributor

Closed with #215

@ErinBecker ErinBecker reopened this May 14, 2019
@ErinBecker
Copy link
Contributor

Actually, it looks like this didn't quite work. I'm re-opening the issue and making a note to myself to look into this tomorrow.

@ErinBecker ErinBecker self-assigned this May 14, 2019
@ErinBecker
Copy link
Contributor

I haven't been able to figure out how to get this to work so am thinking of putting the two different options in "solutions" boxes like are used on the setup page. I'll put in a PR with this option today.

@ErinBecker
Copy link
Contributor

Ok, so upon closer inspection, the reason this isn't working is because there isn't any content in the Unix section, so when the toggle is switched over to Unix, the display doesn't change. I think @ACharbonneau had mentioned populating this content at one point, but I may be misremembering.

@ErinBecker ErinBecker mentioned this issue Jun 1, 2019
Closed
@ACharbonneau
Copy link
Contributor

That's weird, I did fix it. Maybe that bit got lost in the PRs? Let me see if I can re-fix it

@ACharbonneau ACharbonneau mentioned this issue Jun 1, 2019
Merged
@akshayparopkari akshayparopkari removed the status:in progress Contributor working on issue label Jun 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ErinBecker ErinBecker

Labels
help wanted Looking for Contributors
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ACharbonneau @aschuerch @akshayparopkari @ErinBecker