Sign into GBS Warning

[mostaphaRoudsari]

"- User should be prompted to sign into GBS if they haven’t yet. o Currently, the warning for RunAnalysis.RunEnergyAnalysis if I forget to sign in is not helpful."

This looks like an issue with Null ProjectId and not login check but we need to test it again.

[eertugrul]

It seems the RunId returns null and we try to cast RunID as Integer value while de-serialization. I have add conditional check if response returns null, don't deserialize, if not null serialize and return RunId.

I like to able to add warning message when it go wrong instead throw an exception and fail to whole process. I will ask Dynamo team if it is possible.

[bhowes-tt]

@eertugrul - let's have another look at this one. It looks some defense on the ProjectId input is all that's needed.

[eertugrul]

@mostaphaRoudsari I couldn't repeat the problem !

[bhowes-tt]

I could, but it's a dynamo problem. The issue arises when a null is passed into the ProjectId input - our code never even executes because Dynamo can't cast a null to an Int.