-
Notifications
You must be signed in to change notification settings - Fork 88
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
[Business Partner KIT] Release 24.08 #988
[Business Partner KIT] Release 24.08 #988
Conversation
Thank you for your contribution. *Note: There is a chance, that the IP check won't be required. The openAPI plugin will be discontinued and the API and the respective MDX-files needs to be removed, please see the following guidance: #970 After the removal of the APIs maybe you won't have more than 1000 lines anymore. |
@jSchuetz88, Yeah, you rightly said after successful migration to API hub, we won't have more that 1000 lines anymore. |
My bad, you are right.
Yes, the removal is mandatory as we prepare to update docusaurus, which is currently blocked by the API plugin. As this PR is technically already compliant to TRG 1.08, only the MDX-files have to be removed. @danielmiehle After the removal of the MDX-files, this PR is good to go from my side. |
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.
The changes made here are all good.
However, since this pull request should reflect all updates for 24.08 I propose additional changes in the Operators View to remove outdated or confusing information. I created an issue for that here #990
Question is just if we want to address these issues in this pull request or create a new one @SujitMBRDI
We can adapt operation view in same pull request. |
@nicoprow I have adapted changes for operation view mentioned in #990. Now, recent commit streamlines both BPDM repository information and information on present Business Partner KITs. |
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 the Operation View had still some good parts we can overtake. But the content looks good now and is up-to-date
Description
In this pull request, we have updated changes in business partner kit related to release 24.08.
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: