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

bug修复、shaRelated调整 #1563

Merged
merged 2 commits into from
Jul 11, 2024
Merged

bug修复、shaRelated调整 #1563

merged 2 commits into from
Jul 11, 2024

Conversation

PZ157
Copy link
Contributor

@PZ157 PZ157 commented Jul 11, 2024

PR受影响的平台

所有平台

PR描述

乔公、吴国太姓名适配;
shaRelated标签重置;
更改【灵怒】【百鸣】写法;
为没有【】的杀的技能描述补充【杀】

扩展适配

1、姓名适配
(1)对于去掉前缀后的武将名翻译,如果他是单姓+名,如“贾诩”、“张春华”,无需作额外改动;
(2)如果您的武将包使用的lib.character[i]仍然是旧写法(数组形式),请在lib.character[i][4]中添加字符串"name:姓氏|名字",没有姓氏/名字的地方置null。如夏侯氏"name:夏侯|null"、貂蝉"name:null|null"。此外多头武将也需要额外补充,每个人名用"-"分隔。如卫温诸葛直"name:卫|温-诸葛|直",黄巾雷使"name:null|null-null|null",以此类推。
(3)如果您使用的是lib.character[i]={sex:"male",group:"qun",hp:3,skills:["jiang"]}这种新写法,只需要额外加一个names:"姓氏|名字",即可,规则同上(不需要"name:"了)。
2、shaRelated标签
默认将技能描述中含“【杀】”的技能视为与杀相关的技能,您可以在技能中添加shaRelated:false标签取消之;
同样地,对于技能描述中没有“【杀】”的技能但您认为此技能与【杀】相关,您可以在技能中添加shaRelated:true标签加入之;
技能描述中只含“杀”的不算,如“杀死”;
当然这是便于大家直接获取此类技能的一个便捷标签,出于各种原因您也可以不加

检查清单

  • 我已经进行了充足的测试,且现有的测试都已通过
  • 如果此次PR中添加了新的武将/新的语音文件,则我已在character/rank.js中添加对应的武将强度评级/在lib.translate中加入语音文件的文字台词
  • 如果此次PR涉及到新功能的添加,我已在PR描述中写入详细文档
  • 如果此次PR需要扩展跟进,我已在扩展适配中写入详细文档
  • 如果这个PR解决了一个issue,我在诱因和背景中明确链接到该issue
  • 我保证该PR中没有随意修改换行符等内容,没有制造出大量的Diff
  • 我保证该PR遵循项目中.editorconfigeslint.config.mjsprettier.config.mjs所规定的代码样式,并且已经通过prettier格式化过代码

@PZ157 PZ157 changed the title bug修复、shaRelated调整【进行中】 bug修复、shaRelated调整 Jul 11, 2024
@Spmario233 Spmario233 merged commit bd86a5e into libccy:PR-Branch Jul 11, 2024
@Spmario233 Spmario233 mentioned this pull request Jul 23, 2024
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