radiegtya / meteoris2

a Realtime Javascript Boilerplate base on Meteor Js Framework
MIT License
246 stars 46 forks source link

Link to preliminary copy of new doc version #51

Closed martinhbramwell closed 9 years ago

martinhbramwell commented 9 years ago

Please criticise the documentation heavily.

martinhbramwell commented 9 years ago

Glad you like it.

Did you see my remarks in Trello?

radiegtya commented 9 years ago

yes, I've seen it. But I think the discussion better continues at github. The other thing was in Your documentation, You aren't give people know where is job.name (the job object) are from, like in the google docs (which is on the collection helpers part). I think that is very important.

martinhbramwell commented 9 years ago

Hi Ega

I'm not certain which bit of text you mean.

Could you paste into a reply email the text from google docs that I skipped over?

Thanks, Martin

radiegtya commented 9 years ago

in this part, please search it at google docs

on the above code We can clearly see that we loop through "jobs" (#each jobs). So we need a helper to get the jobs collection. Open up "client/view/employees/_form.js" and add this code.

You should at the image below it. it was an important part.

martinhbramwell commented 9 years ago

Ok. I have now looked at that.

I removed it from the documentation because the code is there automatically from the code generation. There is no code to add.

On Tue, Apr 7, 2015 at 10:06 AM, Ega Wachid Radiegtya < notifications@github.com> wrote:

in this part, please search it at google docs

on the above code We can clearly see that we loop through "jobs" (#each jobs). So we need a helper to get the jobs collection. Open up "client/view/employees/_form.js" and add this code.

— Reply to this email directly or view it on GitHub https://github.com/radiegtya/meteoris/pull/51#issuecomment-90571424.