Skip to content

fix: correct DOCUMENT_ALREADY_EXISTS and DOCUMENT_REPLACE_DIFFERENT_DOCID error message wording #2547

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.

Sign up for GitHub

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

Open
adityaanikam wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: correct DOCUMENT_ALREADY_EXISTS and DOCUMENT_REPLACE_DIFFERENT_DOCID error message wording #2547

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/main/resources/errors.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -344,15 +344,15 @@ request-errors:
# DOCUMENT request errors
- scope: DOCUMENT
code: DOCUMENT_ALREADY_EXISTS
title: Document the given _id already exists
title: Document with the given _id already exists
body: |-
Cannot insert the document: a document already exists with given '_id' (${id}).

Either resend the command with distinct '_id', or use an update command instead.

- scope: DOCUMENT
code: DOCUMENT_REPLACE_DIFFERENT_DOCID
title: Document the given _id already exists
title: Document with the given _id already exists
body: |-
The replace document and document resolved using filter have different '_id's: ${replaceId} (replace document) vs. ${matchedId} (document that filter matches).

Expand Down