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

Interstellar exoplanets 翻译更新 #2199

Closed
wants to merge 10 commits into from
Closed

Interstellar exoplanets 翻译更新 #2199

wants to merge 10 commits into from

Conversation

FuBangkun
Copy link

@FuBangkun FuBangkun commented Jul 13, 2022

  • 我已仔细阅读贡献指南 CONTRIBUTING
  • 我已确认标题符合{模组英文全名} {简述}的格式,如Ender IO 翻译新增/更新/修正/删除
  • 我已确认英文原文(如 en_us.json)存在且完整,内容与中文对应;
  • 我已确认提交文件的路径名称正确例子);
    • 如果是 1.12 翻译,应该是:projects/1.12.2/assets/{CurseForge 项目名称}/{ModID}/lang/zh_cn.lang
    • 如果是 1.16 及以上的翻译,应该是:projects/{版本}/assets/{CurseForge 项目名称}/{ModID}/lang/zh_cn.json
  • 我已阅读并同意按 CC BY-NC-SA 4.0 协议发布我的作品;
  • 刷新 PR 的标签/状态,有需要再点击;

@boring-cyborg boring-cyborg bot added 1.12.x Minecraft 版本标签 Forge 包含 Forge 模组 labels Jul 13, 2022
@cfpa-bot
Copy link
Contributor

cfpa-bot bot commented Jul 13, 2022

模组 🔗 链接 🎨 相关文件
Interstellar: Exoplanets 🔍 源代码   | 🟩 MCMOD | 📁 对比
最新模组文件1.12.2/🅱 Interstellar-Exoplanets-1.12.2-0.1.2
1.12.2/🅱 Interstellar-Exoplanets-1.12.2-0.1.3.0
语言文件链接1.12.2/zh_cn.lang
1.12.2/en_us.lang

💾 基于此 PR 所打包的资源包:


exoplanets Mod Domain 验证通过。
ℹ exoplanets-1.12.2 语言文件验证通过。
ℹ exoplanets-1.12.2 模组内语言文件验证通过。


  • 🔄 勾选这个复选框来强制刷新

@boring-cyborg
Copy link

boring-cyborg bot commented Jul 13, 2022

  • 没有完工的汉化仍可提交pr,但请设置为draft。
  • 不要望文生义,有些看起来十分离谱的东西可能是正确的。
  • 尽量做到边开游戏边翻译。
  • 请确认翻译能够正常使用并按照规范格式提交。
  • 善用更新、提交等词语。提交汉化文件时请一并上传英文原文。提交英文原文时请一并上传空白中文文件(1.12的文件为无内容的文件,1.16的文件为为只包含左右花括号的文件,例如 此处 )。

@cfpa-bot cfpa-bot bot added the 100+ 行数标签 label Jul 13, 2022
@mamaruo
Copy link
Member

mamaruo commented Jul 13, 2022

#2072 我建议没有中译的星体保留英文

/diff

@cfpa-bot
Copy link
Contributor

cfpa-bot bot commented Jul 13, 2022

@mamaruo 回复内容过长,已经上传至https://gist.github.com/84b6382e1f15c81d913c0cb71dd36d78

galaxy.wormhole=虫洞

# Messages
information.dev.exoplanets:beta.1=继续您同意您将要遵守的条款
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里怎么读起来怪怪的

Copy link
Member

@mamaruo mamaruo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

问题有点多

Comment on lines +45 to +46
exoplanets.gui.discordLink=链接
exoplanets.gui.failed=尝试打开链接时出现问题
Copy link
Member

@mamaruo mamaruo Jul 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查级别 备注
🔴 严重 / 阻止合并,刻不容缓 /

discord ≠ 链接……
翻译翻着把原文抛了……

原文:
exoplanets.gui.discordLink=Discord Invite
exoplanets.gui.failed=There was a problem while trying to open Discord Invite Link

tile.exoplanets.deco0.name=光栅
tile.exoplanets.deco1.name=点状光栅
tile.exoplanets.satellite_antenna.name=卫星天线
tile.exoplanets.stand_console.name=控制台支架
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查级别 备注
⚪ 轻微 / 不阻止合并但需三思

这什么东西,存疑

原文:
tile.exoplanets.stand_console.name=Console Stand
tile.exoplanets.metaldecoration.name=Floor Grating

@FuBangkun
Copy link
Author

可以了吗?所有问题都解决了

@mczph
Copy link
Contributor

mczph commented Jul 20, 2022

你搞好的点一下resolved

@mamaruo
Copy link
Member

mamaruo commented Jul 20, 2022

可以了吗?所有问题都解决了

并没有

另外我上次 review 没有看完全部的行,问题肯定还有的

@dovisutu
Copy link
Contributor

dovisutu commented Dec 22, 2022

ping @FuBangkun

@dovisutu
Copy link
Contributor

dovisutu commented Jan 5, 2023

  • 7+7时限进入第二阶段。从该评论发出的时间开始计时。

ping @FuBangkun

@dovisutu dovisutu added the 需要更改 在合并之前需要做出更改 label Jan 5, 2023
@FuBangkun FuBangkun closed this by deleting the head repository Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.12.x Minecraft 版本标签 100+ 行数标签 Forge 包含 Forge 模组 需要更改 在合并之前需要做出更改
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants