fpdcc / ccfp-asset-dashboard

CCFP Asset Dashboard
0 stars 1 forks source link

Persist portfolios, part 1 #98

Closed hancush closed 2 years ago

hancush commented 2 years ago

Overview

This PR:

There will be merge conflicts as a result of some file renaming and minor changes in #93, but we'll deal with that later.

TODO

Testing Instructions

hancush commented 2 years ago

@smcalilly I think we can keep the data table views the same for now, since the BaseDataTableView is doing some extra work for us that I don't want to reimplement in DRF.

hancush commented 2 years ago

@smcalilly Once this secures your approval, can I merge into master, would you prefer to wait to merge / deploy to staging until the full UI is done? (I have another changeset chilling on a separate branch.)