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

[platform] j***s 提交组件改动:gzpd.com #813

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

Conversation

mip-audit-platform[bot]
Copy link
Contributor

No description provided.

@@ -0,0 +1,221 @@
import './index.less'
Copy link
Collaborator

Choose a reason for hiding this comment

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

这类组件可以直接利用 mip-list 实现,另外弹窗目前有 mip-modal 官方组件进行支持,无需重复开发

https://www.mipengine.org/v2/docs/interactive-mip/data-driven-and-dom-render.html

Choose a reason for hiding this comment

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

不写组件没办法监听复制事件吧

Copy link

@lyb765428743 lyb765428743 Sep 28, 2019

Choose a reason for hiding this comment

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

里面还有付费后的回调域,然后根据链接上带的order参数判断状态设置cookie,这些都做不到吧

Choose a reason for hiding this comment

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

我重新上传了

Copy link
Collaborator

Choose a reason for hiding this comment

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

那提交的组件应该只包含对复制监听的扩展,其余的部分都可以用 mip现成机制实现

Choose a reason for hiding this comment

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

那提交的组件应该只包含对复制监听的扩展,其余的部分都可以用 mip现成机制实现

写个人组件就是要将这功能写在一块啊。这在结构那写一半,组件写一点也太麻烦了吧

@mipengine-bot
Copy link
Member

构建成功,请点击链接获取编译代码或预览效果

Built with commit 13e3a12

https://deploy-preview-813--mip2-extensions-platform-preview.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.

4 participants