Notification Service needs strict Checkstyle

Description

The Notification service doesn't have checkstyle set to strict like the Requisition service and Reference Data services do. We agreed that strict checkstyle was better and so this service needs checkstyle to be set to strict and the checkstyle issues resolved:

http://build.openlmis.org/job/OpenLMIS-notification-service/

Acceptance criteria:

Notification Service has strict checkstyle enabled and matches the checkstyle document in Requisition Service
Current checkstyle issues are resolved

Linked work items

relates to

OLMIS-1184

Template Service needs strict checkstyle

QAlity Plus - Test Management

Checklists

Activity

Show:

Josh Zamor

October 21, 2016 at 7:39 PM

https://openlmis.atlassian.net/browse/OLMIS-1184#icft=OLMIS-1184 should likely be done first - the checkstyle warnings here may be sourced from cloning the template service.

Resize issue view side panel

Done

Pinned fields

Click on the next to a field label to start pinning.

Details

Assignee

Paweł Gesek

Reporter

Josh Zamor(Deactivated)

Story Points

Original estimate

Time tracking

2h 30m logged

Components

Notification

Sprint

None

Fix versions

3.0

Priority

Minor

Parent

OLMIS-1094 Consolidate notifications (daily digest), customize messages, and support SMS

Time Assistant

Created October 21, 2016 at 7:36 PM

Updated March 1, 2017 at 8:45 PM

Resolved October 28, 2016 at 2:14 PM