Feat/mip custom novel #1273
Merged
Feat/mip custom novel #1273
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
部署成功，请访问地址获取编译代码。 Built with commit b839ad8 |
huanghuiquan
reviewed
Nov 6, 2018
src/mip-custom/mip-custom.js
Outdated
| var novelInstance = isRootPage ? currentWindow.MIP.novelInstance : currentWindow.parent.MIP.novelInstance | ||
| var adsCache = novelInstance.adsCache || {} | ||
| var me = this; | ||
| var novelAds = adsCache.adStategyCacheData.template; |
| var adsCache = novelInstance.adsCache || {} | ||
| var me = this; | ||
| var novelAds = adsCache.adStategyCacheData.template; | ||
| if (novelAds) { |
src/mip-custom/mip-custom.js
Outdated
| var novelAds = adsCache.adStategyCacheData.template; | ||
| if (novelAds) { | ||
| novelAds.map(function (value) { | ||
| value.map(function (ad) { |
huanghuiquan
previously approved these changes
Nov 7, 2018
Ricardo-Li
approved these changes
Nov 8, 2018
hehexiaoxue
approved these changes
Nov 8, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
提交pr时需要关联issue，请大家遵守
#1272
1、升级点 （清晰准确的描述升级的功能点）
配合小说的广告计算展现不同广告
2、影响范围 （描述该需求上线会影响什么功能）
已经做小流量命中小流量，有影响
3、自测 checklist
chrome
4、需要覆盖的场景和case
5、自测机型和浏览器
温馨提示：非 mip-extensions 仓库的组件请在组件平台提交，否则一律打回；