faizan khan
faizan khan
Hey everyone, I have started working on this issue. First of all, can someone confirm by looking at the following screen recording if I am reproducing the issue correctly. https://www.loom.com/share/daaf5d30f76e455aa46705829fdd739d...
Bringing the discussion from slack in here. Following is the graph created for the chained callbacks app, where (CoV = CountriesValue, CiV= CitiesValue, CiO = CitiesOptions, DiC = DisplayChildren, cb...
@alexcjohnson commented the following Ah OK, I see what you’re saying Faizan. I think your interpretation is correct, but yeah we may need to specifically handle several cases differently: -...
The fix that I have right now is the second one, i.e. `CountriesValue` upon update, initiates two callbacks (`CitiesOptions`, and `DisplayValue`). I traverse the `CitiesOptions` callback path to see if...
In summary the two options that @wbrgss mentioned seem to be the solutions here. 1. Prune the extraneous callback, which is what I am doing right now. 2. Block/hold for...
cc @alexcjohnson.
Hi @groegercesg, this is in the queue, the fix has been finalized, just needs some final testing and rebasing before it can be merged.
cc @ranggakd https://github.com/codecrafters-io/build-your-own-x/pull/812
@fraghag sorry we got busy with the DE5 release. Now that the release is done, I can schedule some time to get this through. Ideally in the next few weeks.
I ll have a look into it , but I dont give you any gurantees cause I have a full time job you know....