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/mip ad ecom height #1260

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Feat/mip ad ecom height #1260

wants to merge 6 commits into from

Conversation

Espoir-L
Copy link
Member

提交pr时需要关联issue,请大家遵守
#1259
1、升级点 (清晰准确的描述升级的功能点)
小说阅读器添加监听load事件

2、影响范围 (描述该需求上线会影响什么功能)
现在小说使用定制化的组件

3、自测 checklist
目前代理本地代码,测试小说从结果页到阅读页内广告正常

4、需要覆盖的场景和case

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

5、自测机型和浏览器

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

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

@mipengine-bot
Copy link
Member

mipengine-bot commented Oct 11, 2018

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

Built with commit a9d7054

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

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.

2 participants