Ticket #65 (closed New feature: fixed)

Opened 4 years ago

Last modified 4 years ago

Lost your password? module

Reported by: matt Owned by:
Priority: major Milestone: Piwik 0.9.9 - Stable release
Component: Core Keywords:
Cc: Sensitive:

Description

When user lost their password they should have the ability to receive it by email.

Attachments

lostpassword.patch Download (6.9 KB) - added by mauser 4 years ago.

Change History

Changed 4 years ago by matt

  • milestone set to Stable release

Changed 4 years ago by mauser

Working on this.

Changed 4 years ago by matt

(In [447]) [mauser]- refs #62 commited huge work by Maciej Zawadzi?\197?\132ski: thanks! still some work to do + QA but looks really good :) [mauser]- refs #116 commited partial fix [mauser]- refs #65 commited partial fix - fixed broken tests

Changed 4 years ago by matt

feedback:

  • I think you didn't test the code when input the email, i fixed a bug there
  • An exception is thrown on my box:

Warning: mail() [function.mail]: "sendmail_from" not set in php.ini or custom "From:" header missing in F:\piwik\svn-dev\trunk\libs\Zend\Mail\Transport\Sendmail.php on line 91 need to check for that and if it doesn't work then point the user to a FAQ that explains how to change the password manually

  • there is copied pasted code in the templates, it should be refactored so that all Login templates use the same structure (we should never reuse code when refactoring is possible)
  • good job :)

Changed 4 years ago by mauser

Changed 4 years ago by mauser

  • check if email was sent
  • code is no longer duplicated in templaes

Changed 4 years ago by matt

  • status changed from new to closed
  • resolution set to fixed

(In [454]) [mauser] - fixes #116, font consistent, escape key now works [mauser] - fixes #65, now catching error, printing friendly message, refactoring template code [mauser] - fixes #62, fixing JS translations api, Piwik is now ready to be translated!!! :)

Note: See TracTickets for help on using tickets.