Skip to content
This repository was archived by the owner on Jul 19, 2019. It is now read-only.

Not need this route in lesson 06 #23

Closed
wants to merge 2 commits into from

Conversation

perfectfoolish
Copy link

In Lesson 06, we did not need this route, it should be added in lesson 08.

The lesson 08 have added it as below snippet.

      {/* add it here, as a child of `/` */}
      <IndexRoute component={Home}/>

Fulei Li added 2 commits February 15, 2016 23:17
In Lesson 06, we did not need this route, it should be added in lesson 08.

The lesson 08 have added it as below snippet.

```
      {/* add it here, as a child of `/` */}
      <IndexRoute component={Home}/>
```
@npverni
Copy link

npverni commented Feb 24, 2016

Dup of #16. NavLink is fixed elsewhere

@ryanflorence
Copy link
Contributor

We changed the repo structure to make way more sense, previously the git history was part of the "structure" and that was a terrible idea, making PRs impossible to merge.

Now each lesson has its own directory so we can just click the merge button!

Mind updating your PR to work with the current code on the master branch? Thanks!

@timdorr
Copy link
Member

timdorr commented Mar 29, 2016

Thanks! This was fixed in #122.

@timdorr timdorr closed this Mar 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants