Acker Dawn Apple

Results 167 comments of Acker Dawn Apple

Build an example of your issue here and then paste us the link: https://kaue.github.io/jsonexport/demo/ Then include annotated screen capture highlighting issue on that page. We need better examples and illustrations...

[Simplified direct link to reproduce issue here](https://kaue.github.io/jsonexport/demo/?json=[{%22name%22:%22Bob%22,%22lastname%22:%22Smith%22,%22family%22:{%22name%22:%22Peter%22,%22type%22:%22Father%22},%22interests%22:{%22music%22:%22jazz%22}},{%22name%22:%22James%22,%22lastname%22:%22David%22,%22family%22:{%22name%22:%22Julie%22,%22type%22:%22Mother%22},%22interests%22:{%22books%22:%22pulp%20fiction%22}},{%22name%22:%22Robert%22,%22lastname%22:%22Miller%22,%22family%22:null,%22location%22:[1231,3214,4214]},{%22name%22:%22David%22,%22lastname%22:%22Martin%22,%22nickname%22:%22dmartin%22,%22interests%22:{%22books%22:%22pulp%20fiction%22}}]&options={}) Ok, I can see what you are calling out in that we need extra commas at the end. Not sure who will get...

@MelissaSnell I just noticed pull request #81 might just be your fix. It has conflicts right now but if worked out I believe it aims to address your issue

@bestekov have you checked out and tried #81? I tried briefly reviewing and I'm too out of context with what's going on to verify. It may help to have you...

[Link to example of current results relevant to claim in description](https://kaue.github.io/jsonexport/demo/?json=%7B%22companies%22:%5B%7B%22display_name%22:%22A%22,%22investments%22:%5B%7B%22name%22:%22B%22%7D,%7B%22name%22:%22C%22%7D%5D%7D,%7B%22display_name%22:%22B%22,%22investments%22:%5B%7B%22name%22:%22D%22%7D,%7B%22name%22:%22E%22%7D%5D%7D%5D%7D&options=%7B%7D) (UPDATE: note to self that issuer has provided updated link example in next comment)

Acknowledged. Due to complexity it will take me another round of reviewing and examining. It would behoove you to attempt a most basic summation if possible for me. It’s taking...

@kaue you bet friend, I will do a review within a max 4 days. Fairly loaded but I know you just asking for an eye ball review. I will respond...

Issue #87 I think is in reference to this. If so lets call it out, resolve merge conflicts, and perhaps include a test if one is not included in this...

Great callouts. I’ll aim to review within 4 days

I promised to review this but made a mistake and reviewed another PR #90 thinking it was this one A changelog change was just made and it drew my attention...