Broken on premise :(

[aneillans]

Unfortunately #10 breaks on premise usage of the module. Need to consider a different way for identifying whether the user has entered a FQDN (i.e. local) TFS server, or means a VSTS account.
Ironically, possibly the best method would be just looking for a slash (/) as this wouldn't be in an account name, but would be guaranteed to be present in an on-premise url.

Thoughts?

[aneillans]

"Fix" for it in #12 I'm not happy with the fix, its hacky, but it works. Needs more thought I think.

[DarqueWarrior]

Do you still have concerns about this fix?

[aneillans]

Yeah, but it generally works. It definitely has a smell about it.

…

Sent from my iPhone

On 20 Sep 2017, at 18:06, Donovan Brown ***@***.***> wrote: Do you still have concerns about this fix? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[DarqueWarrior]

Feel free to submit a PR for a better fix. I have not had any issues so I am going to close this. Thanks.