Conversation
Collaborator
Author
|
CDS Report manager-unit-test#389.0 ✘
|
Collaborator
Author
|
CDS Report manager-unit-test#390.0 ✘
|
Collaborator
Author
|
CDS Report manager-unit-test#391.0 ✘
|
Collaborator
Author
|
CDS Report manager-unit-test#392.0 ✘
|
Collaborator
Author
|
CDS Report manager-unit-test#393.0 ✘
|
MaximeBajeux
requested review from
Tsiorifamonjena,
aTurmo,
aottr,
fredericvilcot,
lolottetheclash and
sachinrameshn
July 1, 2026 13:58
Collaborator
Author
|
CDS Report manager-unit-test#394.0 ✘
|
ref: #INC0298894 Signed-off-by: Guillaume Hyenne <guillaume.hyenne@ovhcloud.com>
Signed-off-by: ovh-cds <opensource@ovh.net>
buildSubmitBody started from the whole contact and overwrote every field listed in the configuration rule, including the ones hidden because they are not applicable to the extension. resolveFormValue turns null into '', so updating the owner of a .eu domain sent nationality as an empty string and the API answered [owner.nationality] "" does not match the enum. Skip the fields that have no value, like the legacy AngularJS controller which only merged the fields actually rendered in the form. ref: #INC0254501 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Kylian Bainvel <kylian.bainvel@ovhcloud.com>
web-domains was the only app declaring react-query as a caret range (^5.90.2). The CI installs with `pnpm install --no-lockfile`, so the range resolved to the latest release at build time: 5.102.0 dropped `promise` from QueryObserverBaseResult and tsc started failing on a mock that was written against 5.90.x: serviceList.spec.tsx(83,7): error TS2353: Object literal may only specify known properties, and 'promise' does not exist in type '... & InfiniteQueryObserverSuccessResult<...>' Pin react-query to 5.90.12 and the devtools to 5.91.1, the exact versions the other apps already use. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Kylian Bainvel <kylian.bainvel@ovhcloud.com>
ref: #DCE-346 Signed-off-by: Atef ZAAFOURI <atefzed@gmail.com> Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Signed-off-by: ovh-cds <opensource@ovh.net>
Collaborator
Author
|
CDS Report manager-unit-test#476.0 ✘
|
… is missing The API returns accessUrl as a bare hostname, so the anchor was resolved as a relative path by the browser and opened a 404 inside the Manager instead of the VSPC console. Signed-off-by: maelthi <mael-thi.seme.ext@ovhcloud.com> Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Signed-off-by: ovh-cds <opensource@ovh.net>
ref: #TAPC-7268 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
Signed-off-by: ovh-cds <opensource@ovh.net>
Collaborator
Author
|
CDS Report manager-unit-test#477.0 ✘
|
ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
…ge block ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
…the dashboard ref: #TAPC-7203 Signed-off-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
Signed-off-by: ovh-cds <opensource@ovh.net>
Collaborator
Author
|
CDS Report manager-unit-test#478.0 ✘
|
ref: #PUWEBPT-465 Signed-off-by: Emilien Razafimampiandra <emilien.razafimampiandra.ext@corp.ovh.com>
…on on multisite ref: #DCE-392 Signed-off-by: Emilien Razafimampiandra <emilien.razafimampiandra.ext@corp.ovh.com>
ref: #DCE-393 Signed-off-by: Emilien Razafimampiandra <emilien.razafimampiandra.ext@corp.ovh.com>
…tabase on my site ref: #DCE-392 Signed-off-by: Emilien Razafimampiandra <emilien.razafimampiandra.ext@corp.ovh.com>
Signed-off-by: ovh-cds <opensource@ovh.net>
Collaborator
Author
|
CDS Report manager-unit-test#479.0 ✘
|
Collaborator
Author
|
CDS Report manager-unit-test#480.0 ✘
|
Collaborator
Author
|
CDS Report manager-unit-test#481.0 ✘
|
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automerge master in develop