search

cerebral-legacy / cerebral-website

Cerebral website
http://cerebral-website.herokuapp.com
8 stars 14 forks source link

type error in markdown/nesting.md #31

Closed henri-hulski closed 8 years ago

henri-hulski commented 8 years ago

// We just add the chains that puts the application in // "messagesOpened" state messageOpened: [...messagesOpened, ...messageOpened]

and

When the user goes to /messages/123 the messagesOpened and messageOpened chain will run.

looks a little strange to me.

At least it's hard to reason about ;)

Guria commented 8 years ago

Could you clarify more? What exactly here is issue? Does it about array spread operator?

henri-hulski commented 8 years ago

It was about 3 times using messageOpened. But maybe I just don't understand it.

henri-hulski commented 8 years ago

Ok, I read it another time and actually it's messageOpened and messagesOpened. This makes somehow sense. So sorry for this.

christianalfoni commented 8 years ago

Though you make a good point @henri-hulski , we should change it to avoid confusion. Very easy to misread that