Skip to content

Conversation

Aschen
Copy link
Contributor

@Aschen Aschen commented Dec 20, 2018

What does this PR do?

Change parameters order for collection:create to put optional param in last position

How should this be manually tested?

  • Step 1 :
  • Step 2 :
  • Step 3 :

@codecov-io
Copy link

codecov-io commented Dec 20, 2018

Codecov Report

Merging #348 into 6-dev will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##            6-dev     #348   +/-   ##
=======================================
  Coverage   96.56%   96.56%           
=======================================
  Files          27       27           
  Lines        1397     1397           
=======================================
  Hits         1349     1349           
  Misses         48       48
Impacted Files Coverage Δ
src/controllers/document.js 94.85% <100%> (ø) ⬆️

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 8716be7...e8a9599. Read the comment docs.

@alexandrebouthinon alexandrebouthinon merged commit 27fa579 into 6-dev Dec 21, 2018
@alexandrebouthinon alexandrebouthinon deleted the fix-document-create-param-order branch December 21, 2018 13:40
@Aschen Aschen mentioned this pull request Dec 27, 2018
Aschen added a commit that referenced this pull request Dec 27, 2018
Bug fixes

    [ #349 ] [build] Update webpack build config to fix raw web usage (Njuelle)
    [ #345 ] Fix 343 unhandled rejection on connect (Aschen)

Others

    [ #348 ] Put document:create optional args in last position (Aschen)
@Aschen Aschen mentioned this pull request Jan 7, 2019
Aschen added a commit that referenced this pull request Jan 7, 2019
#### Bug fixes

- [ [#349](#349) ] [build] Update webpack build config to fix raw web usage   ([Njuelle](https://github.com/Njuelle))
- [ [#345](#345) ] Fix 343 unhandled rejection on connect   ([Aschen](https://github.com/Aschen))

#### New features

- [ [#353](#353) ] Add method to use custom controllers in the SDK   ([Aschen](https://github.com/Aschen))

#### Enhancements

- [ [#350](#350) ] Have host param mandatory for protocols   ([Aschen](https://github.com/Aschen))

#### Others

- [ [#348](#348) ] Put document:create optional args in last position   ([Aschen](https://github.com/Aschen))
---
@scottinet scottinet mentioned this pull request Jan 17, 2019
scottinet pushed a commit that referenced this pull request Jan 17, 2019
Change parameters order for collection:create to put optional param in last position
@scottinet scottinet mentioned this pull request Jan 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants