Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants