Skip to content

Jinho ted#1

Open
CCallahanIV wants to merge 3 commits intocodefellows-seattle-301d14:masterfrom
jmpaik:jinho-ted
Open

Jinho ted#1
CCallahanIV wants to merge 3 commits intocodefellows-seattle-301d14:masterfrom
jmpaik:jinho-ted

Conversation

@CCallahanIV
Copy link
Copy Markdown

Single-line Summary

Today, Jinho and Ted paired together. It took about 30 minutes

Reflect and summarize on your process for each TODO item :

With Jinho as Driver:

  • We used the results from our class api call to fill out the handlebars template.
  • We double checked the script calls in the bottom of index.html

With Ted as Driver:

  • We used the class $.ajax api call as a template to finish the requestRepos method. We made sure to invoke the callback function INSIDE the ajax method (due to its asynchronous nature).

Checklist (before submitting, fill in each set of square brackets with an 'x')

  • We have titled the Pull Request similar to our branch name (ex: 'brian-rick').
  • This PR includes commits from both myself and my partner; e.g. We followed good pair programming practices by switching driver/navigator roles.
  • There is no extraneous, unrelated code included in this PR.
  • We have summarized our TODO: process above.

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.

2 participants