Modified http://127.0.0.1:7085/igbStatusCheck to return
"var igbVersion = 9.1.6"
instead of
"var isIgbRunning=True"

[~aloraine] Please check the postman api response to check the sample response and let me know if it is fine.

May need to double-check anything that hits this endpoint (bar/galaxy) to see if they are looking for a 200 response code or if they are actually validating the response (var isIgbRunning=True).

Nowlan Freese
I have double-checked both AppStore and galaxy, bioviz to check if they are validating whether IGB is running based upon the response. It seems they are just checking the status 200 to detect wether igb is running or not. I have tested that on my local machine by browsing both appstore and galaxy to open files using the IGB version which does not return isIGBRunning and tested, it seems to be working fine.

Request for Chaitanya Kintali:

• Could you remove the spaces flanking the = (equals) character?
• Please provide a link to your branch for code review.

Code Branch with spaces removed:
https://bitbucket.org/ckintali/igb-ckintalifork/branch/IGBF-2546

Please submit PR when ready.

attn: Chaitanya Kintali

Pull Request Link:
https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/830/igbf-2546/diff

Since, Pull Request is merged, Moving to Needs Testing.

I see an extra import that is not necessary - import static com.affymetrix.common.CommonUtils.IGB_NAME;
We can remove this!
[~aloraine] -What do you suggest?

Otherwise the code looks good and works fine.

Yes, I agree. Thank you for catching this!
I have already merged it, though.
Noor Zahara - Would you add a new commit that deletes the unneeded import? What do you think?

I can do that as a part of another ticket.

Closing the ticket.

Sure - that sounds great. Please make the ticket and proceed at your convenience.

To-do:

• Please confirm that the un-needed import has been removed. If not, remove it and/or confirm that the import is required.
• Please use Postman or other REST API tool to confirm that IGB statusCheck returns the version

Note: We have already confirmed that other applications that use this pre-existing endpoint are OK with the change. Note that the change is that the 'var' previously was "True" and did not indicate the version.