-
Notifications
You must be signed in to change notification settings - Fork 26
MTM-62651 Added release notes for withChildren default value change #3157
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
base: develop
Are you sure you want to change the base?
MTM-62651 Added release notes for withChildren default value change #3157
Conversation
Preview available here |
...s/cumulocity-20-25-113-0_MTM-62651_default-value-for-the-inventory-withChildren-parameter.md
Outdated
Show resolved
Hide resolved
…TM-62651_default-value-for-the-inventory-withChildren-parameter.md
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.
This is a breaking change, can we making it more obvious in the Change Log? Will this go into API Change?
@JanePorter Yes it is. It was announced in https://drive.google.com/file/d/1vukBH7i_8yT-lLcH41k_zYoVdeFArlTs/view?usp=sharing. |
|
...s/cumulocity-20-25-113-0_MTM-62651_default-value-for-the-inventory-withChildren-parameter.md
Outdated
Show resolved
Hide resolved
…TM-62651_default-value-for-the-inventory-withChildren-parameter.md
@pkepa-c8y Please change the type to API change then. |
@BeateRixen there is lable:
whet else need to be changed ? |
...s/cumulocity-20-25-113-0_MTM-62651_default-value-for-the-inventory-withChildren-parameter.md
Outdated
Show resolved
Hide resolved
...s/cumulocity-20-25-113-0_MTM-62651_default-value-for-the-inventory-withChildren-parameter.md
Outdated
Show resolved
Hide resolved
The change type should be "API change" instead of
The change type should be API change" instead of fix. Just done. |
...s/cumulocity-20-25-113-0_MTM-62651_default-value-for-the-inventory-withChildren-parameter.md
Outdated
Show resolved
Hide resolved
…TM-62651_default-value-for-the-inventory-withChildren-parameter.md Co-authored-by: Dariusz Kaczyński <100694039+kaczynskid-sag@users.noreply.github.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.
As per a conversation with the R&D team we will be holding off this change until we have been able to get further feedback on the impact to our own UI and the customer's custom UIs
...s/cumulocity-20-25-113-0_MTM-62651_default-value-for-the-inventory-withChildren-parameter.md
Outdated
Show resolved
Hide resolved
@pkepa-c8y What is actually the status of this change? Still on hold? |
@BeateRixen Yes, there is still no decision whether we can merge it. |
No description provided.