OLMIS-4833: added endpoint to retrieve pending email verification

Activity

FEOLMIS-3051 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 33m 4 I updated the OLMIS-4907 to make sure that endpoints woul...
    Reviewer - Complete 9m 1 One idea I have is making it hasPendingEmailVerification ...
    Reviewer - Complete 9m 1 add changelog
    Reviewer - Complete 10m 2 I don't quite understand what this API is for?
    Total   1h 1m 8  
    #permalink

    Objectives

    This endpoint is needed for user notification on the UI

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Mateusz Kwiatkowski

    add changelog

    add changelog

    /src/.../web/UserControllerIntegrationTest.java Changed
    /src/.../domain/EmailVerificationToken.java Changed
    /src/.../dto/EmailVerificationTokenDto.java Added 1
    /src/.../service/EmailVerificationNotifier.java Changed
    /src/main/.../service/PermissionService.java Changed
    /src/main/.../auth/web/UserController.java Changed 5
    /src/.../migration/20180612131415248__add_email_field_to_verification_email.sql Changed
    /src/.../schemas/emailVerificationToken.json Added
    /src/main/resources/api-definition.yaml Changed
    /src/.../dto/EmailVerificationTokenDtoTest.java Added
    Open in IDE #permalink
    /src/.../service/EmailVerificationNotifierTest.java Changed
    /src/.../service/PermissionServiceTest.java Changed
    /src/.../auth/EmailVerificationTokenDataBuilder.java Changed
    /CHANGELOG.md Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against