-
Notifications
You must be signed in to change notification settings - Fork 213
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.
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
[doc] Add 3.0-version migration docs #337
Conversation
docs/api/3.0-migration.md
Outdated
@@ -215,3 +215,39 @@ Due to a bug where `originalUrl` was being assigned not as the root url, but the | |||
* Parameter `originalUrl` from `resolveWrappers` has been correctly renamed to `previousUrl`: `resolveWrappers(ad, wrapperDepth, previousUrl)` | |||
* `resolveAds` param `originalUrl` in `options` object has been renamed to `previousUrl`. It also can be passed `url` into `options`. For example: `resolveAds(ads = [], { url, previousUrl })` | |||
* `options` parameter in `VASTParser` `parse` function now expects `previousUrl` instead of `originalUrl`. It also can be passed `url` into `options`. | |||
|
|||
#### Improvement for vast parser to trigger warnings event logs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this isn't actually a breaking change, because it is in a util file not exported in the module
The fixes for Windows in #340 require node 10.12+ so perhaps you could add a word that min version of node is changed. |
5212b0c
to
05b4f73
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small missing spaces otherwise, good job 💯
Co-authored-by: Françon Clément <clement.francon@dailymotion.com>
Co-authored-by: Françon Clément <clement.francon@dailymotion.com>
Description
To help migrating from version 2 to version 3.
Type