TIY-Durham / 2015-FALL-FEE

Class repo for the Front-End Engineering class of the Fall 2015 cohort at TIY Durham
18 stars 12 forks source link

17 -- Camille Mauban #327

Closed cmauban closed 9 years ago

cmauban commented 9 years ago
cmauban commented 9 years ago

Journal, Week 4+

Last night you brainstormed topics. Now it's time to choose two -- one for your reflective entry and one for your tutorial. Outline both entries, then get them reviewed by your peers. Be sure to review theirs, too!

Reading APIs: GitHub

Tonight, focus on the following questions about the GitHub API, which will be important as you build out your profile:

GitHub Revolution, Part 2

Yaks shaved, Skitching complete, now to start making the HTML and (S)CSS for your profile template. For the purposes of this exercise, omit the page header with the search field and primary navigation.

Start by creating a new branch named release--0.0.1 from master in your TIY-GitHub repository. All your work from tonight will be in feature branches that stem from and merge back into this one, which represents your first, tiny, incremental release version: 0.0.1

Octocat in Blue

The profile page is generally laid out into two uneven columns that are centered on the page:

Octocat in Blue... Columns

In a new branch named feature--basic-layout based on release--0.0.1, start with some MF'g HTML and (S)CSS to get those outermost layout elements correct: a set of two proportionate but uneven columns centered on the page. The contents are unimportant to begin with; just focus on the overall layout first, i.e. the blue rectangles.

Open a PR for this work into the release--0.0.1 branch and solicit peer review. Get 2x :+1: (and some commentary) before you merge it. Locally, check out release--0.0.1 and git pull to update it.

Just the Facts, Cat... Octopus... Thing.

Next, focus on filling in the "details" column with information for Octocat. Create another new branch based on release--0.0.1 after the PR for feature--basic-layout is merged. Now you're building "features": small, self-contained pieces of the overall functionality.

Just the facts, cat

Isn't all of that information available from the API data you saved last night? You bet it is! Identify the fields in octocat.json that correspond to the source data for each area and document it in a Skitch note in specs/ for now.

Just looking at this, you should probably have some questions, like:

Push this branch to GitHub and open a PR as soon as you have a commit, merge it when you're finished (enough) and have solicited a code review from two of your peers (that means they left you comments and gave you a :+1:). Then git pull the updates into your local release--0.0.1 branch. What happens when you push that branch up to GitHub? Don't you have a PR for that already?

Gimme the Dataz!

We might as well practice writing some jQuery while we're here. In another feature branch -- feature--fetch-data -- use jQuery.ajax to fetch the data from your octocat.json file and log it to the console. That code should go inside the main.js file, enclosed in an IIFE.

Like before, push this feature branch to GitHub and open a PR into the release--0.0.1 branch. When your PR is reviewed, merge it.

BEAST MODE

There's a correlation between the data in octocat.json and the values in the left column, correct? Can you select each element that corresponds to a data field and replace its contents with the correct value from the JSON file?

cmauban commented 9 years ago

Journal: Week 4 https://github.com/cmauban/cmauban.github.io/pull/10

Assignments https://github.com/cmauban/TIY-Assignments/pull/21

GitHub- release into master: https://github.com/cmauban/TIY-GitHub/pull/3 basic layout into release: https://github.com/cmauban/TIY-GitHub/pull/2 profile details into release: https://github.com/cmauban/TIY-GitHub/pull/4 fetch data into release: https://github.com/cmauban/TIY-GitHub/pull/5

my comments on GitHub: https://github.com/Ricecoder/TIY-GitHub/pull/2#issuecomment-144177436 https://github.com/ShayHall/TIY-GitHub/pull/2#issuecomment-144176045 https://github.com/ericksonk/TIY-GitHub/pull/2 https://github.com/rosalsm/TIY-GitHub/pull/3 https://github.com/ericksonk/TIY-GitHub/pull/4 https://github.com/Ricecoder/TIY-GitHub/pull/5 https://github.com/nickycadavillo/TIY-GitHub/pull/5

my comments on Journals: https://github.com/ShayHall/shayhall.github.io/pull/10 https://github.com/karjac14/karjac14.github.io/pull/11 https://github.com/s4dmw/s4dmw.github.io/pull/17 https://github.com/jeremyrist/jeremyrist.github.io/pull/23 https://github.com/drumslayert/drumslayert.github.io/pull/18

al-the-x commented 9 years ago

Please review and close...