Skip to content

Conversation

@1110101
Copy link

@1110101 1110101 commented Mar 10, 2025

The second parameter of ManagedObject#bindProperty only accepts a object.
The third (and forth) parameters are undocumented and should not be used.

https://ui5.sap.com/#/api/sap.ui.base.ManagedObject%23methods/bindProperty

This change removes text which describes the old usage and replaces it with the new one. (For example, passing the formatter in oBindingInfo.formatter)

  • some minor ES6 improvements

@cla-assistant
Copy link

cla-assistant bot commented Mar 10, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@1110101 1110101 closed this Mar 10, 2025
@1110101 1110101 deleted the patch-1 branch March 10, 2025 13:30
@KlattG KlattG self-assigned this Mar 27, 2025
@KlattG KlattG added contribution Valuable Contribution size/m Requires medium effort to get changed. labels Mar 27, 2025
@KlattG
Copy link
Contributor

KlattG commented Mar 27, 2025

Fixed as of UI5 1.135

@sap-doc-bot
Copy link

sap-doc-bot bot commented Mar 27, 2025

Thank you for your valuable content contribution, @1110101! So that we can recognize your contribution in the SAP Community, please check your SAP Community user ID (this is a number) in your personal settings page and share it with us in a reply to this comment. Make sure you just include the number in the reply.

Your user ID is displayed as follows:

Change display name for User ID N

where N is your user ID. For example, 53 is the user ID of the user 'qmacro'.

Please note that we are currently refactoring our profile and badge system on the SAP Community, and will start assigning badges again when that's complete.

@KlattG KlattG added type/other Other documentation problem. and removed size/m Requires medium effort to get changed. labels Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution Valuable Contribution type/other Other documentation problem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants