search

Towdium / JustEnoughCharacters

Universal tweaks to let mods search in Chinese Pinyin
MIT License
77 stars 23 forks source link

[1.18] 添加新生魔艺抄写台魔符解锁界面的支持 #119

Closed Tidy-Bear closed 8 months ago

vfyjxf commented 8 months ago

1.18分支已经过时，请将修改提交到multiple-version/generate.yaml

Tidy-Bear commented 8 months ago

才看到有多版本（ 那我关掉其它两个 PR，统一提到 multiple-version ？

1.18 与 1.19 一致，签名如下 com.hollingsworth.arsnouveau.client.gui.book.GlyphUnlockMenu:onSearchChanged(Ljava/lang/String;Lcom/hollingsworth/arsnouveau/client/gui/book/GlyphUnlockMenu$Filter;)V

1.20 发生变动，移除了Filter，签名如下 com.hollingsworth.arsnouveau.client.gui.book.GlyphUnlockMenu:onSearchChanged(Ljava/lang/String;)V

是将两个版本共同放入generate.yaml 中，还是将 1.20 的单独放到 1.20.1/forge/platform_generate.yaml ？

vfyjxf commented 8 months ago

才看到有多版本（ 那我关掉其它两个 PR，统一提到 multiple-version ？

1.18 与 1.19 一致，签名如下 com.hollingsworth.arsnouveau.client.gui.book.GlyphUnlockMenu:onSearchChanged(Ljava/lang/String;Lcom/hollingsworth/arsnouveau/client/gui/book/GlyphUnlockMenu$Filter;)V

1.20 发生变动，移除了Filter，签名如下 com.hollingsworth.arsnouveau.client.gui.book.GlyphUnlockMenu:onSearchChanged(Ljava/lang/String;)V

是将两个版本共同放入generate.yaml 中，还是将 1.20 的单独放到 1.20.1/forge/platform_generate.yaml ？

其实都没问题，全扔generate.yaml也行的，不影响。

Tidy-Bear commented 8 months ago

merged into PR https://github.com/Towdium/JustEnoughCharacters/pull/122