Skip to content

Conversation

@fdddf
Copy link
Contributor

@fdddf fdddf commented Sep 8, 2024

#212 提交代码,请审核

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

关联ACTION

library_fdddf_diff存在错误:

当前package缺乏必要文件:
 x 缺少component代码
 x 缺少component声明
 x 缺少logic代码


资产共建大赛提交内容中原有的截图,现升级为依赖库使用说明文档。现交内容包括
1. 依赖库代码本身;2. readme代码说明文档;3. 依赖库使用文档说明。
其中readme面向开发者,使用文档面向使用者。
请自己阅读提交作品路径说明。
https://github.com/netease-lcap/CodeWaveAssetCompetition2024/blob/main/%E8%B5%84%E4%BA%A7%E5%85%B1%E5%BB%BA%E5%A4%A7%E8%B5%9B%E6%8F%90%E4%BA%A4%E5%86%85%E5%AE%B9%E8%AF%B4%E6%98%8E.md

参考作品https://github.com/netease-lcap/CodeWaveAssetCompetition2024/tree/main/demo_bjz_JiazhenBao

tips:README.md和依赖库使用文档说明.docx需要保持文件名称不动。后续我们会对提交的PR自动校验目录结构~

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2024

关联ACTION

library_fdddf_diff存在错误:

当前package缺乏必要文件:
 x 缺少component代码
 x 缺少component声明
 x 缺少logic代码


资产共建大赛提交内容中原有的截图,现升级为依赖库使用说明文档。现交内容包括
1. 依赖库代码本身;2. readme代码说明文档;3. 依赖库使用文档说明。
其中readme面向开发者,使用文档面向使用者。
请自己阅读提交作品路径说明。
https://github.com/netease-lcap/CodeWaveAssetCompetition2024/blob/main/%E8%B5%84%E4%BA%A7%E5%85%B1%E5%BB%BA%E5%A4%A7%E8%B5%9B%E6%8F%90%E4%BA%A4%E5%86%85%E5%AE%B9%E8%AF%B4%E6%98%8E.md

参考作品https://github.com/netease-lcap/CodeWaveAssetCompetition2024/tree/main/demo_bjz_JiazhenBao

tips:README.md和依赖库使用文档说明.docx需要保持文件名称不动。后续我们会对提交的PR自动校验目录结构~

@fdddf
Copy link
Contributor Author

fdddf commented Sep 11, 2024

github ci脚本未兼容前端lcap0.5.2或0.6.0 创建 3.8以上的版本目录结构

@github-actions
Copy link
Contributor

关联ACTION

library_fdddf_diff存在错误:

Command failed: npm run usage
npm ERR! Missing script: "usage"
npm ERR! 
npm ERR! To see a list of scripts, run:
npm ERR!   npm run

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2024-09-24T06_52_48_784Z-debug-0.log

@github-actions
Copy link
Contributor

@lyjamare
Copy link
Collaborator

#212 提交代码,请审核

您好,经功能验收同学验收后,反馈如下,烦请按照反馈进行修改:

组件有关于背景色和字体色的4个标题不够语义化,例如删除文本,添加文本,没办法直观得理解什么是删除文本,什么是添加文本。得靠使用者自己试了之后才知道,建议该成更容易让使用者理解的标题名字。

组件行对比确实可以实现,但是字符对比和单词对比两个功能使用起来的效果跟行对比一摸一样,并没有什么明显区别。

该依赖库的功能完全可以靠组件实现,不太明白3个逻辑有什么功能,文档上看,就是看一下对比的信息。但是文档并没有对返回值做一个清晰的说明,建议举个例子,对返回值各个属性做一个详细说明。

@github-actions
Copy link
Contributor

@fdddf
Copy link
Contributor Author

fdddf commented Sep 30, 2024

#212 提交代码,请审核

您好,经功能验收同学验收后,反馈如下,烦请按照反馈进行修改:

组件有关于背景色和字体色的4个标题不够语义化,例如删除文本,添加文本,没办法直观得理解什么是删除文本,什么是添加文本。得靠使用者自己试了之后才知道,建议该成更容易让使用者理解的标题名字。

组件行对比确实可以实现,但是字符对比和单词对比两个功能使用起来的效果跟行对比一摸一样,并没有什么明显区别。

该依赖库的功能完全可以靠组件实现,不太明白3个逻辑有什么功能,文档上看,就是看一下对比的信息。但是文档并没有对返回值做一个清晰的说明,建议举个例子,对返回值各个属性做一个详细说明。

文案已更新,有些可能比较难以表述,参数可以参考这个jsdiff库的官方文档。
那三个逻辑暴露出来是希望 开发者可以根据需要使用,本身和组件使用不冲突或依赖。

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@lyjamare
Copy link
Collaborator

功能及代码已验收,待评分

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