Skip to content

Commit

Permalink
bedrock: fix protocol ItemStackRequest optional type, add 1.21.50 upd…
Browse files Browse the repository at this point in the history
…ated skin (#962)

* Remove times_crafted to optional

This would cause Relay to crash since it did not have the field in v1.21.50.

* Updated steve.json for v1.21.50

* Update types.yml

* Update dataPaths.json

* Remove times_crafted v1.20.30 from v1.21.30 & v1.21.40 data
  • Loading branch information
yowzaoppo authored Jan 7, 2025
1 parent 636df6a commit 6e12664
Show file tree
Hide file tree
Showing 8 changed files with 2 additions and 16 deletions.
4 changes: 0 additions & 4 deletions data/bedrock/1.21.30/protocol.json
Original file line number Diff line number Diff line change
Expand Up @@ -2899,10 +2899,6 @@
"name": "recipe_network_id",
"type": "varint"
},
{
"name": "times_crafted",
"type": "u8"
},
{
"name": "filtered_string_index",
"type": "li32"
Expand Down
1 change: 0 additions & 1 deletion data/bedrock/1.21.30/types.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1074,7 +1074,6 @@ ItemStackRequest:
if optional:
# For the cartography table, if a certain MULTI recipe is being called, this points to the network ID that was assigned.
recipe_network_id: varint
times_crafted: u8
# Most likely the index in the request's filter strings that this action is using
filtered_string_index: li32
if craft_grindstone_request:
Expand Down
4 changes: 0 additions & 4 deletions data/bedrock/1.21.42/protocol.json
Original file line number Diff line number Diff line change
Expand Up @@ -2904,10 +2904,6 @@
"name": "recipe_network_id",
"type": "varint"
},
{
"name": "times_crafted",
"type": "u8"
},
{
"name": "filtered_string_index",
"type": "li32"
Expand Down
1 change: 0 additions & 1 deletion data/bedrock/1.21.42/types.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1078,7 +1078,6 @@ ItemStackRequest:
if optional:
# For the cartography table, if a certain MULTI recipe is being called, this points to the network ID that was assigned.
recipe_network_id: varint
times_crafted: u8
# Most likely the index in the request's filter strings that this action is using
filtered_string_index: li32
if craft_grindstone_request:
Expand Down
4 changes: 0 additions & 4 deletions data/bedrock/1.21.50/protocol.json
Original file line number Diff line number Diff line change
Expand Up @@ -2904,10 +2904,6 @@
"name": "recipe_network_id",
"type": "varint"
},
{
"name": "times_crafted",
"type": "u8"
},
{
"name": "filtered_string_index",
"type": "li32"
Expand Down
1 change: 1 addition & 0 deletions data/bedrock/1.21.50/steve.json

Large diffs are not rendered by default.

1 change: 0 additions & 1 deletion data/bedrock/latest/types.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1078,7 +1078,6 @@ ItemStackRequest:
if optional:
# For the cartography table, if a certain MULTI recipe is being called, this points to the network ID that was assigned.
recipe_network_id: varint
times_crafted: u8
# Most likely the index in the request's filter strings that this action is using
filtered_string_index: li32
if craft_grindstone_request:
Expand Down
2 changes: 1 addition & 1 deletion data/dataPaths.json
Original file line number Diff line number Diff line change
Expand Up @@ -2346,7 +2346,7 @@
"effects": "pc/1.17",
"protocol": "bedrock/1.21.50",
"windows": "bedrock/1.16.201",
"steve": "bedrock/1.20.80",
"steve": "bedrock/1.21.50",
"blocksB2J": "bedrock/1.21.0",
"blocksJ2B": "bedrock/1.21.0",
"proto": "bedrock/latest",
Expand Down

0 comments on commit 6e12664

Please sign in to comment.