search

opensrp / openmrs-module-team

Other
1 stars 5 forks source link

Code Review - REST API #36

Closed maimoonak closed 7 years ago

maimoonak commented 7 years ago

Rest resources should be in package rest Rest resources should not be named a .*RS. Use RequestResource or Resource as postfix Rest resources should NOT include any integer id like logId, teamRoleId etc Rest resources should include REF for properties which are nested objects Rest resources should have both, default and full, representations Rest resources should have auditInfo as openmrs defines it Rest resources search methods should be refactored.

maimoonak commented 7 years ago

All resolved except point 1. i.e. rest resources are not in package rest