Permalink
Browse files

Use new events for password reset KPI (closes #37)

  • Loading branch information...
1 parent c87c891 commit 685d63f89f6d4bed43f3a926e4301454ed6f914b @nmalkin committed Aug 21, 2012
Showing with 4 additions and 8 deletions.
  1. +3 −3 server/config/config.json
  2. +1 −5 server/lib/data.js
@@ -28,9 +28,9 @@
[ "4 - Logged in (assertion generated)", "assertion_generated" ]
],
"password_reset": [
- [ "1 - Enter email", "screen.set_password" ],
- [ "2 - Choose new password", "screen.check_registration" ],
- [ "3 - Email verified", "generate_assertion" ],
+ [ "1 - Begin password reset", "screen.reset_password" ],
+ [ "2 - Choose new password", "user.reset_password_staged" ],
+ [ "3 - Email verified", "user.reset_password_confirmed" ],
[ "4 - Logged in (assertion generated)", "assertion_generated" ]
]
},
View
@@ -201,11 +201,7 @@ exports.passwordResetSteps = function(datum) {
var steps = [];
var events = eventList(datum);
- if( (events.indexOf('screen.check_registration') === -1) ||
- (events.indexOf('screen.set_password') === -1) ||
- (events.indexOf('user.email_staged') !== -1) ||
- (events.indexOf('user.user_staged') !== -1) )
- { // not a password reset
+ if(events.indexOf('screen.reset_password') === -1) { // not a password reset
return steps;
}

0 comments on commit 685d63f

Please sign in to comment.