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.


---

Reply via email to