OLMIS-4833: Fixed problem with sending verification on new email address

Activity

FEOLMIS-3066 8

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 8m 4 in 99% situations both will have value or null
    Reviewer - Complete 2m    
    Reviewer - Complete 5m 4 1% is enough to have a separate message https://review.op...
    Total   15m 8  
    #permalink

    Objectives

    With old code we always send verification email on the same email even if user updated it by UI/API. Now we retrieve existing token details to get a correct email address.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /src/.../web/UserControllerIntegrationTest.java Changed
    /src/main/.../auth/web/UserController.java Changed 8

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against