Initialize a breakpoint as loading#7644
Conversation
|
Presumably we'd need to set this to |
|
@darkwing No dice. Maybe I should leave this for someone more knowledgeable than me |
|
@AnshulMalik or @bomsy, could you take a look at this? I've tried this in the past and ran into the same issues. |
| "hidden": false, | ||
| "id": "foo", | ||
| "loading": false, | ||
| "loading": true, |
There was a problem hiding this comment.
loading is false in src/actions/breakpoints/index.js, but all the following snapshots have loading is true. Could this be the reason for the failing tests?
There was a problem hiding this comment.
It was failing when loading was false in the snapshot. The circle test was failing as well
|
This is more complex than I thought it would be, I'll try to look into it tonight. |
|
Any progress here @AnshulMalik ? |
|
Yeah, there seems to be some issue with breakpoints not persisting aka pending breakpoints. |
5 participants
Fixes #6771
Latest attempt to initialize loading as true