That's why the 'Jorah Fedora' meme pisses me off, like that's all he is, some pathetic dude. This guy isn't Samwell here. He was doing a damn good job with his advice, and for a very specific reason: Where all of Dany's council has had particular experiences (a kingsguard, a slave, a mereneese noble etc), but struggle to communicate and get one solid message across, Jorah on the other hand has seen it all. He's been part of a large Westerosi house. He has fought wars. He has sold slaves and been a criminal. He has been caught and disgraced, and known exile. Now travelled huge amounts on both continents he has. He knows shitloads about every culture they come across. He was always the guy who 'saw both sides' and interpreted things accordingly, from the moment he joined up with Dany and would constantly explain Dothraki culture to help her out, but do it in a Westerosi way that she'd understand.
The guy is smart, he can fight, he has experience of almost the entire world and gives common sense advice based not on honour but on what will advance Dany's goals. He's not afraid to be honest, either. Yes it's awkward as hell that the dude fell in love with her and that he started as a spy, but she needs to forget that, put him in his place, set boundaries and then find a way to work with him. I mean, shit, you won't find anyone more loyal at this stage.