Skip to content
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

feat: 广告合并-bugfix #314 #1287

Merged
merged 4 commits into from
Nov 15, 2018
Merged

feat: 广告合并-bugfix #314 #1287

merged 4 commits into from
Nov 15, 2018

Conversation

Espoir-L
Copy link
Member

提交pr时需要关联issue,请大家遵守

1272

1、升级点 (清晰准确的描述升级的功能点)
广告合并bugfix-为防止config中带的地址有多余的domain域名前缀
2、影响范围 (描述该需求上线会影响什么功能)
使用了custom的页面
3、自测 checklist
chrome
4、需要覆盖的场景和case

  • 是否覆盖了sf打开mip页
  • 是否验证了极速服务mip页面效果

5、自测机型和浏览器

  • 是否覆盖了ios系统手机
  • 是否覆盖了安卓系统手机
  • 是否覆盖了iphone版式浏览器(比如qq、uc、chrome、safari、安卓自带浏览器)
  • 是否覆盖了手百

温馨提示:非 mip-extensions 仓库的组件请在组件平台提交,否则一律打回;

@mipengine-bot
Copy link
Member

部署成功,请访问地址获取编译代码。

Built with commit ca4b902

https://deploy-preview-1287--mip-extensions.netlify.com

@mipengine-bot
Copy link
Member

mipengine-bot commented Nov 15, 2018

部署成功,请访问地址获取编译代码。

Built with commit 87540ff

https://deploy-preview-1287--mip-extensions.netlify.com

@Espoir-L
Copy link
Member Author

关联到issue为:小说节约广告成本 #1272

for (var key in config.paths) {
if (config.paths.hasOwnProperty(key)) {
config.paths[key] = config.domain + config.paths[key];
var customConfig = JSON.parse(JSON.stringify(config));
Copy link
Member

Choose a reason for hiding this comment

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

这是啥操作?为什么需要stringify然后在parse?

Copy link
Member Author

Choose a reason for hiding this comment

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

深克隆这个对象,并且赋给新的一个变量

@Espoir-L Espoir-L merged commit 9325ce8 into master Nov 15, 2018
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.

5 participants