PrintHtml icon indicating copy to clipboard operation
PrintHtml copied to clipboard

Add optional -json for STDOUT JSON string

Open Nineapsofttech opened this issue 7 years ago • 3 comments

Add optional -json for STDOUT JSON string. STDOUT Object of

  • success array - array of success urls.
  • error array - array of fail to load urls.

Nineapsofttech avatar Mar 20 '18 11:03 Nineapsofttech

I like this change, but please move QString dataPath = QApplication::applicationDirPath() back to it's original location below it's comment (much better to define stuff where it is needed, with related comments).

Also please fix all the formatting where the formatting has been compressed as that is now how the code is formatted in this project. So this:

if(this->json){

needs to be come this:

if (this->json) {

Quite a lot of code got reformatted so that all needs to be cleaned up.

kendallb avatar Mar 20 '18 17:03 kendallb

This code tree no longer merges clean, due to other changes in master. Can you resolve the conflicts so I can bring it in?

Also last time we tried Qt5 we ended up with problems (don't recall exactly what they were), but we went back to Qt 4.8 to resolve that. Is Qt5 working OK for you?

kendallb avatar Aug 27 '19 16:08 kendallb

I still have conflicts and can't merge this in clean. Can you resolve them in this PR?

kendallb avatar Sep 10 '19 16:09 kendallb