Skip to content

Comments

Ednick#5

Open
ejdelrio wants to merge 4 commits intocodefellows-seattle-301d21:masterfrom
ejdelrio:ednick
Open

Ednick#5
ejdelrio wants to merge 4 commits intocodefellows-seattle-301d21:masterfrom
ejdelrio:ednick

Conversation

@ejdelrio
Copy link

Single-line Summary

**Today, Nick and Eddie paired together. It took about 1 hour and 20 minutes

Reflect and summarize on your process for each TODO item :

  1. Forked and cloned the repo.
  2. Wrapped our Js files in IFFEs :D
  3. Replaced loops with map and reduce methods
  4. Created a new HandleBar template
  5. Created Author stats object.
  6. Incorporated new function calls into HTML files
  7. Added connString for new pg Client.

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.

ejdelrio and others added 4 commits June 26, 2017 10:25
Refatcored a function to use a fat arrow function and map instead of forEach
Added handlebars template to ArticleView.js
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