Mjz/15 spike topic filter

MAPC / metro-mayors-housing-task-force

Static Site for Metro Mayor's Task Force on Housing

https://housingtaskforce.mapc.org/

MIT License

1 stars 0 forks source link

Mjz/15 spike topic filter #26

Closed mzagaja closed 6 years ago

mzagaja commented 6 years ago

@iangmhill This is my progress so far on making topics and sub topics act more like a filter. Currently it is randomly removing all topics from the Array of selected topics instead of just the clicked one so that is an annoying bug I could use a pair of eyes on. Then we just need to get the sub topics working followed by best practices.

ericyoungberg commented 6 years ago

@mzagaja @iangmhill I finished the filter in a state-reducer fashion, ready for review and merge

iangmhill commented 6 years ago

I think we're moving around a lot of data in the constant setState calls than we have to, but this looks pretty good for now.

ericyoungberg commented 6 years ago

Yeah, I wasn't sure about setting substate so I just kind of took the whole structure and set it each time haha