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

新增农耕工艺材质 #81

Merged
merged 8 commits into from
Nov 22, 2024
Merged

新增农耕工艺材质 #81

merged 8 commits into from
Nov 22, 2024

Conversation

haiman233
Copy link
Contributor

json文件是用脚本生成的,测试的应该是没问题的

@haiman233
Copy link
Contributor Author

b7bb90040120f0430fdac7b0f32927f0
成品是这个

@haiman233
Copy link
Contributor Author

但是现在有一个问题,就是里面有一个物品是草,草的id在1.20.3从grass改成了short_grass,所以,这个有关草的材质在1.20.3以上加载不出来。不确定是要额外再补充一个使用材质id为short_grass的配置,还是要改其它的

@haiman233
Copy link
Contributor Author

同样的还有海龟壳,只是暂时没有以海龟壳为材质的物品

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

不過這個缺少 config 就是了

@haiman233
Copy link
Contributor Author

ok了,所以草的那个id问题该咋解决

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

ok了,所以草的那个id问题该咋解决

只能以最新版為基礎吧 畢竟我的生成器也在最新版 1.21.1

@haiman233
Copy link
Contributor Author

行吧

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

然後這是新的附加 所以要分配一下範圍

@haiman233
Copy link
Contributor Author

2300000-2300100

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

喔怕 你直接指定 23 開頭喔 那也可以
你覺得要分配個一千個還是一萬會比較好

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

一百太少了 怕附加突然又多了一堆奇奇怪怪的神奇道具

@haiman233
Copy link
Contributor Author

emm,250个吧

@haiman233
Copy link
Contributor Author

毕竟是sefi写的附属,我相信他不会有大改动

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

好 那就 300 好了 然後之後新增的附加會以 2300300+100 就是 2300400 開始 預留額外空間

@xMikux
Copy link
Owner

xMikux commented Nov 3, 2024

都用整數

@haiman233
Copy link
Contributor Author

所以现在应该没啥其它问题了吧

@xMikux
Copy link
Owner

xMikux commented Nov 5, 2024

沒u 等我有時間的時候會檢查一下生成有沒有問題

@xMikux xMikux merged commit 12d45c3 into xMikux:main Nov 22, 2024
1 check passed
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