Change the order of assertion generation to be after the user is asked "is this your computer?"
* This fixes a problem where it is possible that a user takes more than 2 minutes to answer, making the already generated assertion invalid. * A lot of general cleanup of the state machine. * Push all post email selection flows to a new state, "email_valid_and_ready" which decides whether to ask the user the computer ownership status. * Assertions are generated after the "is this your computer" is asked. * doEmailChosen is replaced with doGenerateAssertion - there were multiple calls to generate an assertion, not very DRY. * Fix a problem in network.js where a user who completes user registration should be marked as authenticated. * rename email_chosen.js to generate_assertion.js to clarify its purpose. issue #1460
Showing
- lib/static_resources.js 1 addition, 1 deletionlib/static_resources.js
- resources/static/dialog/controllers/actions.js 2 additions, 12 deletionsresources/static/dialog/controllers/actions.js
- resources/static/dialog/controllers/generate_assertion.js 4 additions, 4 deletionsresources/static/dialog/controllers/generate_assertion.js
- resources/static/dialog/resources/state.js 76 additions, 49 deletionsresources/static/dialog/resources/state.js
- resources/static/dialog/start.js 1 addition, 1 deletionresources/static/dialog/start.js
- resources/static/shared/network.js 5 additions, 0 deletionsresources/static/shared/network.js
- resources/static/test/cases/controllers/actions.js 31 additions, 96 deletionsresources/static/test/cases/controllers/actions.js
- resources/static/test/cases/controllers/generate_assertion.js 1 addition, 1 deletion...urces/static/test/cases/controllers/generate_assertion.js
- resources/static/test/cases/resources/state.js 60 additions, 27 deletionsresources/static/test/cases/resources/state.js
- resources/static/test/cases/shared/network.js 38 additions, 8 deletionsresources/static/test/cases/shared/network.js
- resources/views/test.ejs 2 additions, 2 deletionsresources/views/test.ejs
Loading
Please register or sign in to comment