OLMIS-4833: Handle email update


FEOLMIS-3046 15

Keyboard shortcuts  
  • Summarize the review outcomes (optional)


    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 36m 6 other note: in near future this will be moved to the noti...
    Reviewer - Complete 17m 3 Shouldn't this be done by the referencedata though?
    Reviewer - Complete 21m 3 OK, I saw nothing
    Reviewer - Complete 39m 3 Alright, we'll leave this then.
    Total   1h 53m 15  


    1. when user is created, the reference data part of data will contain email but verified flag will be false
    2. when user is updated and email was not changed we do nothing
    3. when user is updated and email has been changed we set old values for email and verified fields for reference data part of data. The new email address will be set when the user will verify the new email.

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/.../web/UserControllerIntegrationTest.java Changed
    /src/.../domain/EmailVerificationToken.java Changed 2
    /src/main/.../auth/service/UserService.java Changed 2
    /src/main/.../auth/web/UserController.java Changed 9
    /src/.../web/UserSaveRequestValidator.java Changed 2
    /src/.../migration/20180612131415248__add_email_field_to_verification_email.sql Added
    Open in IDE #permalink
    /src/.../domain/EmailVerificationTokenTest.java Changed
    /src/.../service/RequestParametersTest.java Added
    Open in IDE #permalink
    /src/test/.../service/UserServiceTest.java Changed
    /src/.../web/UserSaveRequestValidatorTest.java Changed
    /src/test/.../openlmis/auth/DummyUserDto.java Changed
    /src/.../auth/EmailVerificationTokenDataBuilder.java Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against