> > Small typo in method name, otherwise fine. > > Let me know where so I can fix that.
Yeah sorry, those inlined comments are really hard to find. It's this: bool LocationsFactory::lastUrlNeedsAuthencation() const -> bool LocationsFactory::lastUrlNeedsAuthentication() const -- https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-03/+merge/252219 Your team Ubuntu File Manager Developers is subscribed to branch lp:ubuntu-filemanager-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers More help : https://help.launchpad.net/ListHelp