Skip to content

Conversation

Shiranuit
Copy link
Contributor

@Shiranuit Shiranuit commented May 11, 2021

What does this PR do ?

This PR add the action document:deleteFields and the associated documentation and tests
⚠️ Depends on #632

@Shiranuit Shiranuit self-assigned this May 11, 2021
@Shiranuit Shiranuit linked an issue May 11, 2021 that may be closed by this pull request
Copy link
Contributor

@Aschen Aschen left a comment

Choose a reason for hiding this comment

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

You should add the timeout option from your other PR

Copy link
Contributor

@MathieuVeber MathieuVeber left a comment

Choose a reason for hiding this comment

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

LGTM 😉

@scottinet scottinet changed the title Feature document:deleteFields Add support for the document:deleteFields API action May 18, 2021
@codecov
Copy link

codecov bot commented May 19, 2021

Codecov Report

Merging #636 (0600843) into 7-dev (f0beacc) will increase coverage by 0.00%.
The diff coverage is 75.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##            7-dev     #636   +/-   ##
=======================================
  Coverage   86.15%   86.16%           
=======================================
  Files          34       34           
  Lines        1589     1597    +8     
  Branches      279      281    +2     
=======================================
+ Hits         1369     1376    +7     
- Misses        162      163    +1     
  Partials       58       58           
Impacted Files Coverage Δ
src/controllers/Document.ts 68.75% <75.00%> (+0.27%) ⬆️
src/protocols/WebSocket.ts 80.50% <0.00%> (+0.68%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0beacc...0600843. Read the comment docs.

@@ -0,0 +1,7 @@
try {
const response = await kuzzle.document.deleteFields('nyc-open-data', 'yellow-taxi', 'some-id', ['bar'], {source: true});
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit

Suggested change
const response = await kuzzle.document.deleteFields('nyc-open-data', 'yellow-taxi', 'some-id', ['bar'], {source: true});
const response = await kuzzle.document.deleteFields('nyc-open-data', 'yellow-taxi', 'some-id', ['bar'], { source: true });

@Shiranuit Shiranuit merged commit a967b88 into 7-dev May 27, 2021
@Shiranuit Shiranuit deleted the 587-document-deleteFields branch May 27, 2021 11:34
@Shiranuit Shiranuit mentioned this pull request May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add document:deleteFields
4 participants