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

Address SF Loft Feedback #3

Merged
merged 4 commits into from Jan 31, 2017
Merged

Conversation

jpignata
Copy link
Contributor

Includes changes to the lab guide and code to incorporate learnings from the Wild Rydes workshop run at the SF Loft on 1/26. Changes include:

  • Rename lab 1 to pre-lab and decrement the lab number of each subsequent module
  • Return errors to student as a JavaScript alert on email submission
  • Change the cursor to a pointer when hovering over a button
  • Include screenshot of proper Cognito unauthenticated role setup
  • Update copyright year

@jpignata jpignata force-pushed the jp/updates-from-sfloft-feedback branch from d5e37db to 5bd98fd Compare January 28, 2017 14:43
We're seeing a lot of folks come to the loft to this workshop and
getting stuck on setup steps like installing with AWS CLI. By
designating lab 1 a pre-lab, we can promote this workshop with the
requirement that the pre-lab be completed prior to the workshop to
give students a head start on getting their environment configured.

This has the additional benefit of moving the CloudFront distribution
creation step to be done prior to the workshop starting as that can
sometimes take upwards of 30 minutes to provision.

- Decrement subsequent lab numbers
- Rename images
- Update copyright notice
Students found it confusing that the submit button didn't change the
cursor to a pointer. Unfortunately, I don't have the original CSS
from the designer so this change edits the minified CSS.
Use a JavaScript alert to return an error conspicuously to students if
something isn't properly setup with their DynamoDB table or Cognito
identity pool.

- Add common errors with resolution steps
This step often trips up students so add a screenshot to help them
verify what they've entered matches what's intended.
@jpignata jpignata force-pushed the jp/updates-from-sfloft-feedback branch from 5bd98fd to b64de3c Compare January 29, 2017 18:34
@jpignata jpignata merged commit d0dddcb into master Jan 31, 2017
@jpignata jpignata deleted the jp/updates-from-sfloft-feedback branch April 23, 2017 15:47
justonian added a commit that referenced this pull request Nov 26, 2018
Merging updated Cleanup instructions
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.

None yet

1 participant