Skip to content
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

feat(manager-react-components): Bump ods version 18.3.0 #14080

Merged
merged 5 commits into from
Nov 15, 2024

Conversation

aboungnaseng-ovhcloud
Copy link
Contributor

@aboungnaseng-ovhcloud aboungnaseng-ovhcloud commented Nov 13, 2024

Question Answer
Branch? master
Bug fix? no
New feature? no
Breaking change? no
Tickets Fix 14622
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

Bump ods version 18.3..0 and fix publish documentation

Related

@github-actions github-actions bot added documentation Documentation only changes dependencies Pull requests that update a dependency file feature New feature labels Nov 13, 2024
@aboungnaseng-ovhcloud aboungnaseng-ovhcloud force-pushed the feat/mrc-ods-version-18-3 branch 2 times, most recently from 3a2d6e8 to d716b2b Compare November 14, 2024 07:51
@aboungnaseng-ovhcloud aboungnaseng-ovhcloud marked this pull request as draft November 14, 2024 08:50
@aboungnaseng-ovhcloud aboungnaseng-ovhcloud marked this pull request as ready for review November 14, 2024 09:12
@aboungnaseng-ovhcloud aboungnaseng-ovhcloud force-pushed the feat/mrc-ods-version-18-3 branch 2 times, most recently from 1b189c5 to 775cb39 Compare November 14, 2024 09:36
anooparveti
anooparveti previously approved these changes Nov 14, 2024
chipp972
chipp972 previously approved these changes Nov 14, 2024
tristanwagner
tristanwagner previously approved these changes Nov 14, 2024
tristanwagner
tristanwagner previously approved these changes Nov 14, 2024
ref: MANAGER-14622

Signed-off-by: Alex Boungnaseng <[email protected]>
ref: MANAGER-14622

Signed-off-by: Alex Boungnaseng <[email protected]>
ref: MANAGER-14622

Signed-off-by: Alex Boungnaseng <[email protected]>
ref: MANAGER-14622

Signed-off-by: Alex Boungnaseng <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change of isDisabled to disabled is a breaking change IMO. So, we either need to mention it in Documentation or use the version v3.0.
Or we can ignore this change and keep disabled as is.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes Its a breaking change but we wanted to be align with attribute name of ODS

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, then add it in Documentation.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

ref: MANAGER-14622

Signed-off-by: Alex Boungnaseng <[email protected]>
Copy link

sonarcloud bot commented Nov 15, 2024

@anooparveti anooparveti merged commit 040237b into master Nov 15, 2024
15 checks passed
@anooparveti anooparveti deleted the feat/mrc-ods-version-18-3 branch November 15, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
common dependencies Pull requests that update a dependency file documentation Documentation only changes feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants