Skip to content

[pull] master from DefinitelyTyped:master#2182

Merged
pull[bot] merged 10 commits intoSimenB:masterfrom
DefinitelyTyped:master
Apr 7, 2020
Merged

[pull] master from DefinitelyTyped:master#2182
pull[bot] merged 10 commits intoSimenB:masterfrom
DefinitelyTyped:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 7, 2020

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

peterblazejewicz and others added 10 commits April 6, 2020 19:21
- `height`
- `name`i
- `svg`
- `transform`
- properties are optionali

https://git.io/JvNOS

Closes comments: https://git.io/JvNO9

/cc @mmakrzem

Thanks!
- missing optional `extendTraces` parameter of `extendTraces`
- remove superflous `autoScale2d` type from `ModeBarDefaultButtons`
- add test for optional `maxPoints` parameter
- reformat `ModeBarDefaultButtons` for better readability when
adding/chaning supported types

Fixes comment:
#42882 (comment)

Thanks

/cc @mmakrzem
* Adjusted types for mergeTags property or react-email-editor

* Updated version for react-email-editor types

* Fix lint
* make callbacks nullable

* fix aux tests
* Refactor the types for ChangeStreams for accuracy

* Refined ChangeStreams "updatedFields" because I discovered I was slightly mistaken on how they work

* Refactor ExtractIdType to be more efficient / use fewer steps

* Refactor changestream type defs as per PR feedback

* Clarify the difference between Arguments and Parameters; rename and add comments

* Switch to use ExtractIdType as per PR feedback

* Revert change to ExtractIdTypes as per PR request

* Fix missed case where ExtractIdType should have been used
* series gauge data field

* .

* .

Co-authored-by: Andrea Puccio <andrea.puccio@futuryng.com>
* Add definitions for messages

* Add definition author
@pull pull bot added the ⤵️ pull label Apr 7, 2020
@pull pull bot merged commit 7a2c545 into SimenB:master Apr 7, 2020
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.

9 participants