Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...


IssueNotes Decision
1use api and impl folders

Team agreed on separate top level ap and impl folders. Common impl folders can be at top level ie. not under impl.
any class in api should not depend (i.e. nor import) anything from impl or common folders!!!  

2use feature based packages

Team agreed to use the current 4 services (or 5 depending on inventory)  as feature folders directly under impl and some also under api

The following feature structure emerged during the actual refactoring

  • cmsubscriptions (TBC)
  •  data
    • async
  • datajobs
  • inventory
    • sync
      • lcm
    • trustlevel
3when to use common packages

when in doubt use common package ie e.g. exceptions might be re-used for other features in the future
However just because it is used by another feature doesn't mean it is common. e.g. a Class to check th state of a cm-handle could be used anywhere but still very much belong to the 'inventory' feature

4scope

currently only ncmp-service in scope. 

5plan

team agreed to do trial with 'datajobs' first. Code need to be reviewed by both teams. Once this is completed we will address 'legacy' services

...