-
Notifications
You must be signed in to change notification settings - Fork 393
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
Pinia迁移认领:vuex迁移到pinia #308
Comments
@sudoooooo 你好,认领C端这个可以吗? 1、问卷相关 |
@yoruponder 确认认领,已记录 |
认领 |
@1004801012 确认认领,已记录 |
认领: |
@whitexie 确认认领,已记录 |
认领edit问卷编辑页相关 |
@webaddkevin 确认认领,已记录 |
你好,认领 |
@yoruponder 确认认领,已记录 |
@yoruponder 你好,有个认领feature的用户需要基于迁移版本进行开发。目前只剩下erroInfo就可以全部迁移完成,请问方便先提个PR吗? |
@sudoooooo 大佬好的,今晚搞下提PR,最近这几周工作上业务需求太多了,不好意思了 |
@yoruponder 非常感谢理解 |
认领: |
本次迁移源自vue3升级的后续:#108
如疑问,请听官方解释(jiaobian): 为什么你应该使用 Pinia?
如果你想熟悉工程和pinia用法,或者参与贡献,这是一个比较好的机会。
认领方式
1、留言认领:模块或者模块-功能点 + 预计完成时间
2、官方会将跟进人标记到此Issue等于认领成功
3、基于feature/pinia分支,改造后通过以下检测:
vscode无飘红(安装插件:Vue - Official 和 Eslint)
npm run format
npm run lint
相关功能自测通过
4、提交PR到feature/pinia分支
说明
1、由原来的
store
放到stores
目录2、迁移指南
3、pinia推荐迁移成setup的风格
4、示例:user模块迁移案例
5、阶段性会反向合并develop分支,提交PR前关注可能会存在的冲突
任务列表
B端搭建端
userInfo
hasLogined
loginTime
initialized
bannerList
cooperPermissions
surveyId
schema
schemaUpdateTime
currentEditOne
currentEditStatus
moduleConfig(computed)
formConfigList(computed)
currentEditMeta(computed)
currentEditKey(computed)
spaceMenus
spaceType
workSpaceId
spaceDetail
teamSpaceList
surveyList
surveyTotal
searchVal
selectValueMap
buttonValueMap
listFliter(computed)
listOrder(computed)
C端渲染端
surveyPath
isMobile
enterTime
encryptInfo
questionData
questionSeq
renderData(computed)
voteMap
errorInfo
(若有遗漏欢迎进行补充)
rules
bannerConf
baseConf
bottomConf
dataConf
skinConf
submitConf
formValues
The text was updated successfully, but these errors were encountered: