search

meteor / todos

The example app "Todos", written following the Meteor Guide
Other
535 stars 366 forks source link

Official Example doesn't update to latest version 1.11.x #292

Open MichaelJCole opened 4 years ago

MichaelJCole commented 4 years ago

Hi there, am I missing this on a branch or something? Would you like a PR for this?

meteor update to latest version has a ton of packages held back.

kadira:flow-router has been replaced with https://github.com/VeliovGroup/flow-router in the official docs, but not in this repo.

useraccounts:flow-router requires kadira:flow-router. A fork fixes the dependency here: https://atmospherejs.com/mealsunite/flow-routing-extra - it's confusingly named. https://github.com/VeliovGroup/flow-router/issues/21 (alternative package: nxcong:flow-routing)

I commented out the "ActiveRoute" code in app_body.js. It this this can be replaced with: FlowRouter.current()

I really want to get back into Meteor for client work - it would be easier to sell if the example app was up to date.

Would you take a PR for this?

MichaelJCole commented 4 years ago

LOL, I just saw the announcement for 1.11 was this morning. I've been working on other things :-)

filipenevola commented 4 years ago

Hi @MichaelJCole, a PR is welcome here