Github user necouchman commented on a diff in the pull request: https://github.com/apache/guacamole-client/pull/255#discussion_r172016032 --- Diff: guacamole/src/main/webapp/app/navigation/services/userPageService.js --- @@ -57,12 +57,24 @@ angular.module('navigation').factory('userPageService', ['$injector', * A map of all root connection groups visible to the current user, * where each key is the identifier of the corresponding data source. * + * @param {Object.<String, PermissionSet>} permissions + * A map of all permissions granted to the current user, where each + * key is the identifier of the corresponding data source. + * + * @param {Object.<String, + * * @returns {PageDefinition} * The user's home page. */ - var generateHomePage = function generateHomePage(rootGroups) { + var generateHomePage = function generateHomePage(rootGroups, permissions) { var homePage = null; + var settingsPages = generateSettingsPages(permissions); + + // If we have more than one setting page, return the main home page --- End diff -- Okay, added some more verbiage there.
---