-
Notifications
You must be signed in to change notification settings - Fork 175
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
Update context via config changed #3693
Conversation
…gi into update-context-via-configChanged
Co-authored-by: Ndricim <ndricim.rrapi@sap.com>
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.
Tested and working well. Just two small comments regarding documentation
* Updates the context values for visible iframes and LUI web components. | ||
* Note: the updated context values are not persisted. The developers have to do it on their own. | ||
* @param {Object} ctx - The context to be updated. | ||
* @memberof Configuration |
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.
* @memberof Configuration | |
* @memberof Configuration | |
* @example | |
* Luigi.updateContextValues({someData: 'myContext'}) |
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.
also run npm run docu to regenerate Docu stuff
Fixes #3685