Closed dpasque closed 2 years ago
Got the final version of the app state updated, along with all its selectors and reducers. I also updated the fake flow to interact more with the redux state. So now you can pick issue details and see the tasks update in realtime, and even check those tasks off! 🎉
Next: a whole lot of clean-up and some unit testing
Awesome work, @dpasque! I did a quick glance over the changes and everything seems good (from my limited knowledge 😅). I ran the app and unit tests locally and those work fine as well.
I'm not sure how your schedule is tomorrow (Friday), but would you be willing to meet for another sync session? I'd like to get a better understand of the core changes. These files specifically come to mind:
I don't have much to show for wireframing, but we could also talk about some of the helpful feedback/suggestions we've received recently and how it'll impact the UI for Bugomattic. If you have a lot to wrap up before the meetup, no worries!
What Does This PR Add/Change?
In short, this PR adds redux state (and corresponding selectors and reducers) for the basic app state required for the core Bugomattic flow (pick feature > pick issue type > see tasks).
To do that, this PR...
issueDetails
slice to the redux store -- this is where the info about the issue goescompletedTasks
slice to the redux store -- this is how we track the IDs of tasks that have been completed. This lets us provide a "checkbox" experience to the users.I realize there are a lot of files changed, and I'm so sorry! A lot of that is just logistical clean-up. The core changes are themselves not that big. I will leave comments to draw attention to what changes are meaningful!
Testing Instructions
Run the unit tests:
yarn test:once
.I also added some debugging components that let you play around with the fake flow! To start the app locally, just
yarn
to install dependencies, then runyarn start
.You can compare what you see to the fake JSON document located at
src/api/local-reporting-config-response.json
.