remove items from basic_materials that we don't need #2666

Closed
opened 2022-09-12 20:01:14 +00:00 by flux · 5 comments
Member

e.g. "carbon_steel_bar" and "lead_strip"

e.g. "carbon_steel_bar" and "lead_strip"
flux added the
1. kind/enhancement
2. prio/good first issue
labels 2022-09-12 20:01:23 +00:00

Not in favour. If not for good reasons, we should keep mods as vanilla as they come, to make updates easier. Plus, we might find uses for those - I believe the copper strips found application this way in postboxes

Not in favour. If not for good reasons, we should keep mods as vanilla as they come, to make updates easier. Plus, we might find uses for those - I believe the copper strips found application this way in postboxes
Author
Member

Not in favour. If not for good reasons, we should keep mods as vanilla as they come, to make updates easier. Plus, we might find uses for those - I believe the copper strips found application this way in postboxes

i'm proposing to remove the items via overrides in the integration mod (currently yl_commons) - not to modify the basic_materials mod itself.

at the least, we should add these items to the "not_in_creative_inventory" or "not_in_craft_guide" group.

i think we should add lead to your-land, but that's complicated (see your-land/administration#142).

> Not in favour. If not for good reasons, we should keep mods as vanilla as they come, to make updates easier. Plus, we might find uses for those - I believe the copper strips found application this way in postboxes i'm proposing to remove the items via overrides in the integration mod (currently yl_commons) - not to modify the `basic_materials` mod itself. at the least, we should add these items to the "not_in_creative_inventory" or "not_in_craft_guide" group. i think we should add lead to your-land, but that's complicated (see https://gitea.your-land.de/your-land/administration/issues/142).
Author
Member

hid the unused materials in ae677f0ca2.

this is easy to change if we ever end up w/ uses for them.

hid the unused materials in https://gitea.your-land.de/your-land/yl_commons/commit/ae677f0ca2732563f9271aebd682512f4647a016. this is easy to change if we ever end up w/ uses for them.
flux added the
4. step/ready to QA test
label 2022-11-05 17:56:50 +00:00
AliasAlreadyTaken was assigned by flux 2022-11-05 17:56:55 +00:00
flux removed the
2. prio/good first issue
label 2022-11-08 00:03:26 +00:00
Author
Member

this got deployed, but then basic_materials added more stuff, so there's another update needed:

e1cc6458fa

this got deployed, but then basic_materials added more stuff, so there's another update needed: https://gitea.your-land.de/your-land/yl_commons/commit/e1cc6458fa84e6b3486afee8e281a2d8f1f28f99
AliasAlreadyTaken added this to the 1.1.117 milestone 2022-11-23 11:44:33 +00:00
AliasAlreadyTaken added the
ugh/QA OK
label 2023-01-18 12:09:51 +00:00
flux added
5. result/fixed
and removed
4. step/ready to QA test
labels 2023-01-25 17:29:56 +00:00
AliasAlreadyTaken was unassigned by flux 2023-01-25 17:29:58 +00:00
Author
Member

this is live

this is live
flux closed this issue 2023-01-25 17:30:07 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#2666
No description provided.