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

Pinia迁移认领:vuex迁移到pinia #308

Closed
8 tasks done
sudoooooo opened this issue Jun 24, 2024 · 14 comments
Closed
8 tasks done

Pinia迁移认领:vuex迁移到pinia #308

sudoooooo opened this issue Jun 24, 2024 · 14 comments
Labels

Comments

@sudoooooo
Copy link
Collaborator

sudoooooo commented Jun 24, 2024

本次迁移源自vue3升级的后续:#108
如疑问,请听官方解释(jiaobian): 为什么你应该使用 Pinia?

如果你想熟悉工程和pinia用法,或者参与贡献,这是一个比较好的机会。

认领方式

为保障正常建设,认领1个月内无响应(沟通讨论/技术方案/PR),任务自动释放

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的风格

为什么用setup风格?
从代码风格统一:开源工程的代码风格基本采用了vue3的setup风格
组合式写法相比选项式写法能够检测到未被使用的函数
值得一提的是, 当我们定义一个 setup store 时,几乎可以使用任何组合式函数,因为每一个属性都会被> 自动被辨别为 state 、action 或者 getter,而不需要采用options Store的方案显示的声明,具体使用方式可参考下面的迁移示例

4、示例:user模块迁移案例
5、阶段性会反向合并develop分支,提交PR前关注可能会存在的冲突

任务列表

为保障正常建设,认领1个月内无响应(沟通讨论/技术方案/PR),任务自动释放

B端搭建端

  • 1、user用户相关 @skique ,进展:已完成
    userInfo
    hasLogined
    loginTime
    initialized
  • 2、edit问卷编辑页相关 @webaddkevin,进展:已完成
    bannerList
    cooperPermissions
    surveyId
    schema
    schemaUpdateTime
    currentEditOne
    currentEditStatus
    moduleConfig(computed)
    formConfigList(computed)
    currentEditMeta(computed)
    currentEditKey(computed)
  • 3、list空间 @whitexie ,进展:已完成
    spaceMenus
    spaceType
    workSpaceId
    spaceDetail
    teamSpaceList
  • 4、list列表 @whitexie ,进展:已完成
    surveyList
    surveyTotal
    searchVal
    selectValueMap
    buttonValueMap
    listFliter(computed)
    listOrder(computed)

C端渲染端

  • 1、问卷相关 @yoruponder ,进展:已完成
    surveyPath
    isMobile
    enterTime
    encryptInfo
  • 2、答题页相关 @1004801012 ,进展:已完成
    questionData
    questionSeq
    renderData(computed)
    voteMap
  • 3、错误页相关 @yoruponder ,进展:已完成
    errorInfo

(若有遗漏欢迎进行补充)

  • 4、问卷相关的遗漏补充 @whitexie ,进展:已完成
    rules
    bannerConf
    baseConf
    bottomConf
    dataConf
    skinConf
    submitConf
    formValues
@sudoooooo sudoooooo added the WIP label Jun 24, 2024
@sudoooooo sudoooooo changed the title Pinia Pinia迁移认领:vuex迁移到pinia Jun 28, 2024
@yoruponder
Copy link
Contributor

@sudoooooo 你好,认领C端这个可以吗?

1、问卷相关
surveyPath
isMobile
enterTime
encryptInfo

@sudoooooo
Copy link
Collaborator Author

sudoooooo commented Jul 1, 2024

@sudoooooo 你好,认领C端这个可以吗?

1、问卷相关 surveyPath isMobile enterTime encryptInfo

@yoruponder 确认认领,已记录

@1004801012
Copy link
Contributor

认领
2、答题页相关
questionData
questionSeq
renderData(computed)
voteMap

@sudoooooo
Copy link
Collaborator Author

认领 2、答题页相关 questionData questionSeq renderData(computed) voteMap

@1004801012 确认认领,已记录

@whitexie
Copy link
Contributor

whitexie commented Jul 3, 2024

认领:
3、list空间
spaceMenus
spaceType
workSpaceId
spaceDetail
teamSpaceList
4、list列表
surveyList
surveyTotal
searchVal
selectValueMap
buttonValueMap
listFliter(computed)
listOrder(computed)

@sudoooooo
Copy link
Collaborator Author

确认认领,已记录

@whitexie 确认认领,已记录

@sudoooooo sudoooooo mentioned this issue Jul 3, 2024
@webaddkevin
Copy link
Contributor

认领edit问卷编辑页相关

@sudoooooo
Copy link
Collaborator Author

确认认领,已记录

@webaddkevin 确认认领,已记录

@yoruponder
Copy link
Contributor

你好,认领
3、错误页相关
errorInfo

@sudoooooo
Copy link
Collaborator Author

你好,认领 3、错误页相关 errorInfo

@yoruponder 确认认领,已记录

@sudoooooo
Copy link
Collaborator Author

你好,认领 3、错误页相关 errorInfo

@yoruponder 你好,有个认领feature的用户需要基于迁移版本进行开发。目前只剩下erroInfo就可以全部迁移完成,请问方便先提个PR吗?

@yoruponder
Copy link
Contributor

@sudoooooo 大佬好的,今晚搞下提PR,最近这几周工作上业务需求太多了,不好意思了

@sudoooooo
Copy link
Collaborator Author

@sudoooooo 大佬好的,今晚搞下提PR,最近这几周工作上业务需求太多了,不好意思了

@yoruponder 非常感谢理解

@whitexie
Copy link
Contributor

认领:
问卷相关的遗漏补充
rules
bannerConf
baseConf
bottomConf
dataConf
skinConf
submitConf
formValues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants