Skip to content

Conversation

@Aschen
Copy link
Contributor

@Aschen Aschen commented Jan 28, 2021

What does this PR do?

kuzzleio/kuzzle#1946

@codecov
Copy link

codecov bot commented Jan 28, 2021

Codecov Report

Merging #595 (f46f030) into 7-dev (ab9ff14) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##            7-dev     #595   +/-   ##
=======================================
  Coverage   87.26%   87.26%           
=======================================
  Files          32       32           
  Lines        1468     1468           
  Branches      261      261           
=======================================
  Hits         1281     1281           
  Misses        136      136           
  Partials       51       51           
Impacted Files Coverage Δ
src/Kuzzle.ts 86.63% <0.00%> (ø)
src/controllers/Document.ts 68.47% <0.00%> (ø)

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 ab9ff14...f46f030. Read the comment docs.

Copy link
Member

@alexandrebouthinon alexandrebouthinon left a comment

Choose a reason for hiding this comment

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

Missing Unit tests ?

Copy link
Contributor

@scottinet scottinet left a comment

Choose a reason for hiding this comment

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

minor documentation issues

Comment on lines 37 to 38
| `silent` | <pre>boolean</pre><br/>(`false`) | If `true`, then Kuzzle will not generate notifications <SinceBadge version="change-me"/> |
| `retryOnConflict` | <pre>int</pre><br/>(`0`) | The number of times the database layer should retry in case of version conflict |
Copy link
Contributor

Choose a reason for hiding this comment

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

Options are sorted alphabetically:

Suggested change
| `silent` | <pre>boolean</pre><br/>(`false`) | If `true`, then Kuzzle will not generate notifications <SinceBadge version="change-me"/> |
| `retryOnConflict` | <pre>int</pre><br/>(`0`) | The number of times the database layer should retry in case of version conflict |
| `retryOnConflict` | <pre>int</pre><br/>(`0`) | The number of times the database layer should retry in case of version conflict |
| `silent` | <pre>boolean</pre><br/>(`false`) | If `true`, then Kuzzle will not generate notifications <SinceBadge version="change-me"/> |

Comment on lines 47 to 50
| `refresh` | <pre>string</pre><br/>(`""`) | If set to `wait_for`, waits for the change to be reflected for `search` (up to 1s) |
| `silent` | <pre>boolean</pre><br/>(`false`) | If `true`, then Kuzzle will not generate notifications <SinceBadge version="change-me"/> |
| `source` | <pre>boolean</pre><br/>(`false`) | If true, returns the updated document inside the response |
| `lang` | <pre>string</pre> | Specify the query language to use. By default, it's `elasticsearch` but `koncorde` can also be used. <SinceBadge version="7.4.8"/> |
Copy link
Contributor

Choose a reason for hiding this comment

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

sort:

Suggested change
| `refresh` | <pre>string</pre><br/>(`""`) | If set to `wait_for`, waits for the change to be reflected for `search` (up to 1s) |
| `silent` | <pre>boolean</pre><br/>(`false`) | If `true`, then Kuzzle will not generate notifications <SinceBadge version="change-me"/> |
| `source` | <pre>boolean</pre><br/>(`false`) | If true, returns the updated document inside the response |
| `lang` | <pre>string</pre> | Specify the query language to use. By default, it's `elasticsearch` but `koncorde` can also be used. <SinceBadge version="7.4.8"/> |
| `lang` | <pre>string</pre> | Specify the query language to use. By default, it's `elasticsearch` but `koncorde` can also be used. <SinceBadge version="7.4.8"/> |
| `refresh` | <pre>string</pre><br/>(`""`) | If set to `wait_for`, waits for the change to be reflected for `search` (up to 1s) |
| `silent` | <pre>boolean</pre><br/>(`false`) | If `true`, then Kuzzle will not generate notifications <SinceBadge version="change-me"/> |
| `source` | <pre>boolean</pre><br/>(`false`) | If true, returns the updated document inside the response |

@Aschen Aschen merged commit c70c788 into 7-dev Jan 29, 2021
@Aschen Aschen deleted the add-notify-option branch January 29, 2021 08:17
@Aschen Aschen mentioned this pull request Jan 29, 2021
Aschen added a commit that referenced this pull request Jan 29, 2021
# [7.5.3](https://github.com/kuzzleio/sdk-javascript/releases/tag/7.5.3) (2021-01-29)


#### Enhancements

- [ [#596](#596) ] Error stacktrace overhaul   ([Aschen](https://github.com/Aschen))
- [ [#595](#595) ] Allows to not send realtime notification on document controller   ([Aschen](https://github.com/Aschen))
---
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.

5 participants