Rearrange and split functions in Expenses

codeforboston / cliff-effects

Cliff effects guidance prototype (archived)

https://codeforboston.github.io/cliff-effects/#/

MIT License

30 stars 64 forks source link

Rearrange and split functions in Expenses #965

Closed knod closed 5 years ago

knod commented 6 years ago

For testing and maintenance. Splitting the file itself is an option, but I'm not sure there's a reason to. None of the components need to be used anywhere else. Something like splitting into two scripts so that the name of the file isn't so misleading might worth it, though.

knod commented 6 years ago

Can't be really checked out till #967 is resolved.

knod commented 5 years ago

imo, the diff is probably best viewed one commit at a time.

Also, the styles are messed up right now, but that's because of an unrelated issue that's fixed in another PR.

knod commented 5 years ago

Not WIP anymore