search

mtr-cherish / cherish

http://cherish.meteor.com
MIT License
24 stars 3 forks source link

No forgot password functionality on cherish.meteor.com #114

Closed chrisco23 closed 5 years ago

chrisco23 commented 9 years ago

I don't see any way to recover a password on cherish.meteor.com

iamlasse commented 9 years ago

I think it comes with accounts-base but hasnt been implemented yet. Feel free to add yourself and it will be moved to ready.

chrisco23 commented 9 years ago

On another project I use ian:accounts-ui-bootstrap-3. Was this package considered for cherish at all? It gives you Forgot Password, Login with Facebook, Google, Github, ... things all out of the box. Then, not sure if it's cleanest, but because I'm moving to material design, I use https://github.com/FezVrasta/bootstrap-material-design

chrisco23 commented 9 years ago

Here's the ian:accounts-ui-bootstrap-3 straight out of the box signin_ui-accounts

jeffshaver commented 9 years ago

@chrisco23 i am a personal fan of having a dedicated login/register page instead of a drop down. I am also not a huge bootstrap fan either. Plus, we are already using materialize. We should get "forgot password" functionality added. This seems like a good article on doing this: https://gentlenode.com/journal/meteor-2-build-a-complete-registration-system/3

chrisco23 commented 9 years ago

If we don't use accounts-ui-bootstrap-3, any objection to using accounts-entry package? I really don't see any reason to reinvent the wheel, piecemeal-like. And there are plenty of reasons not to:

  1. good well-tested code already used in production already exists
  2. less work for us
  3. familiar UI/UX for our users

That's one of the perks of Meteor: one doesn't have to build auth from scratch.

jeffshaver commented 9 years ago

@chrisco23 my issue with that package is that it has 86 open issues, 32 pull requests and hasn't been touched in 3 months. But I guess I could be swayed either way. Would like @iamlasse, @mariocatch and everyone elses opinion on this.

chrisco23 commented 9 years ago

Another good candidate, I believe, is useraccounts:materialize, though I've not used it before.

On Thu, Jul 9, 2015 at 7:19 AM Jeffrey E. Shaver II < notifications@github.com> wrote:

@chrisco23 https://github.com/chrisco23 my issue with that package is that it has 86 open issues, 32 pull requests and hasn't been touched in 3 months. But I guess I could be swayed either way. Would like @iamlasse https://github.com/iamlasse, @mariocatch https://github.com/mariocatch and everyone elses opinion on this.

— Reply to this email directly or view it on GitHub https://github.com/mtr-cherish/cherish/issues/114#issuecomment-119998427 .