OLMIS-6879: Added Superset client registration


FEOLMIS-4124 6

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 10m 2 You're right, but I used isNotBlank to prevent empty strings
    Reviewer - 0% reviewed      
    Reviewer - Complete 21m 4 StringUtils.isNotEmpty() would fit there
    Reviewer - 100% reviewed 5m    
    Reviewer - Complete 0m    
    Total   37m 6  


    There are no specific objectives for this review.

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/main/java/.../auth/domain/Client.java Changed
    Open in IDE #permalink
    /src/.../auth/ApiSupersetClientInitializer.java Added 6
    Open in IDE #permalink
    /src/main/resources/application.properties Changed
    Open in IDE #permalink
    /src/.../auth/ApiSupersetClientInitializerTest.java Added
    Open in IDE #permalink
    /src/test/.../auth/ClientDataBuilder.java Changed
    Open in IDE #permalink
    /CHANGELOG.md Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time