Node functionality is split between core and data_set_manager

[hackdna]

• Node model is in data_set_manager/models.py
• Node API (v1 and v2) is in the core Django app:
  • NodeResource is in core/api.py
  • NodeViewSet is in core/views.py
  • NodeSerializer in core/serializers.py
• NodeApiV2Tests is in the data_set_manager/tests.py

This violates Django design principles and makes the code difficult to understand and maintain.

[jkmarx]

It should go in data_set_manager? @hackdna @scottx611x

[scottx611x]

@jkmarx I would say so. It would be much tougher to move the model/migrations into core/

[hackdna]

Also, core Django app should be reduced in size and ideally, just be used for bit and pieces that don't belong to any particular app (Two Scoops 29.1).