fixup #225 - fix logged job link

[MartinNowak]

fixup #250

Description

Checklist

• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have run changie new to create a changelog entry

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, don't hesitate to ping @drewbanin.

CLA has not been signed by users: @MartinNowak

[MartinNowak]

Any chance we can get this reviewed and merged before 1.3.0 @McKnight-42 @jtcohen6

[Fleid]

Hey @MartinNowak, sorry for the delay.
I can't find the reason that motivates the switch, have I missed a conversation somewhere? Thanks!

[VersusFacit]

We'll need some context here to explain why we need to change the order of the if statement. Is this meant to be primarily syntactic? Because shifting the parameters could introduce boolean short-circuit related changes in logic (or perhaps the current order is bugged itself)

Also, is the order of these parameters wrong? Certainly share the context that led to these changes ✌️

[Fleid]

I'll close for now, please re-open if need be!