-
Notifications
You must be signed in to change notification settings - Fork 171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix logout label #1453
Fix logout label #1453
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than 1 question all LGTM! Tested in incognito mode as well
core/src/Authorization.html
Outdated
AuthLayerSvc.setProfileLogoutFn(profileLogoutfn); | ||
} | ||
profileLogoutFnDefined = true; | ||
AuthLayerSvc.setProfileLogoutFn(profileLogoutfn); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think setProfileLogoutFn
should get logoutItem.customLogoutFn
as parameter otherwise profileLogoutfn is not defined. See removed line 176
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
* master: (21 commits) style the mobile avigation for Documentation (#1452) allow hash and section deeplinks (#1447) Add link to blogs (#1442) Release v1.2.4 (#1457) Fix logout label (#1453) adds a check if persisted collapsible state is closed and prevent persistion on resize (#1431) update links (#1445) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (#1414) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (#1414) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (#1414) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (#1414) redirect for blog on landingpage Add blog to website (#1439) release v1.2.3 (#1437) remove non-working code (#1436) Release v1.2.2 (#1435) fix double frame (#1433) Added showLabel attribute to node for top level nav (#1429) Strip Documentation in Changelog generation (#1404) Feature/badge counter (#1409) ...
* feature-globalsearch: (21 commits) style the mobile avigation for Documentation (SAP#1452) allow hash and section deeplinks (SAP#1447) Add link to blogs (SAP#1442) Release v1.2.4 (SAP#1457) Fix logout label (SAP#1453) adds a check if persisted collapsible state is closed and prevent persistion on resize (SAP#1431) update links (SAP#1445) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (SAP#1414) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (SAP#1414) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (SAP#1414) 1381 update version of Fundamental Styles on Fiddle to 0.10.0 (SAP#1414) redirect for blog on landingpage Add blog to website (SAP#1439) release v1.2.3 (SAP#1437) remove non-working code (SAP#1436) Release v1.2.2 (SAP#1435) fix double frame (SAP#1433) Added showLabel attribute to node for top level nav (SAP#1429) Strip Documentation in Changelog generation (SAP#1404) Feature/badge counter (SAP#1409) ...
Fixes #1448