Fix property and event names for popconfirm component - #20240
Merged
Conversation
Member
|
Deploy preview for element ready! Built with commit 9c43018 |
Contributor
Author
|
@iamkun take a look |
This was referenced Oct 29, 2020
|
long time no see |
|
这种mr 也能合并,线上全部确认事件无效了 |
|
@wulucxy element 更新日志里都写了是不兼容升级 , 自己不看就乱升级线上环境 真是头铁 |
|
@xx45 去了解下 npm 版本规则,minor 版本是自动升级的。这种breaking change 放在 minor 版本要害死一堆程序员了 |
|
生产环境锁版本 这不是基本操作 |
|
@xx45 建议多参考业界的实践 |
|
版本控制也不是全部人都跟隨 semver 怕出事就最好用 當然最穩定做法是人手更新 |
|
这次也没有太多的更新,线上项目没有更新了 |
|
建议下次版本升级的时候进行兼容性处理,先同时保留confirm/cancel和onConfirm/onCancel两组事件,然后在3.x版本再将onConfirm/onCancel废弃 |
果然,除了客户端 |
Closed
This file contains hidden or 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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Please make sure these boxes are checked before submitting your PR, thank you!
https://vuejs.org/v2/guide/components-custom-events.html