HEC-1058 json library swap

[meghana-viswanath]

No description provided.

[meghana-viswanath]

@AndyHWChung @priyankadhiman3006 I've added tests for the readers/writes in this commit, can you review them, thanks

[priyankadhiman3006]

Just a question, are we going to merge this now or will do the similar changes in other repos and then merge on the same day?
Cause if this merged and others may break when we try to validate tax code in LB.

[meghana-viswanath]

Just a question, are we going to merge this now or will do the similar changes in other repos and then merge on the same day?
Cause if this merged and others may break when we try to validate tax code in LB.

Good question, I think it's better to do them all together @AndyHWChung ?

[AndyHWChung]

Just a question, are we going to merge this now or will do the similar changes in other repos and then merge on the same day?
Cause if this merged and others may break when we try to validate tax code in LB.

Yeah, we should merge everything together including updates to hec-ui-journey-tests where it's calling our BE test-only endpoint. It'd be a good idea just to clear our mongo databases in the test environments as well once it's merged in

[meghana-viswanath]

Just a question, are we going to merge this now or will do the similar changes in other repos and then merge on the same day?
Cause if this merged and others may break when we try to validate tax code in LB.

Yeah, we should merge everything together including updates to hec-ui-journey-tests where it's calling our BE test-only endpoint. It'd be a good idea just to clear our mongo databases in the test environments as well once it's merged in

I'll keep this in Draft until we are ready to merge

[priyankadhiman3006]

Then again, other code will keep on piling and you have to do rebase and stuff. i think we should divide and conquer :) happy to help