Slot Access Feature Preview #101

Merged
justliliandev merged 7 commits from feature/slot_access into dev/1.19.x 2023-01-30 13:09:58 +00:00
justliliandev commented 2022-12-29 16:18:07 +00:00 (Migrated from github.com)

Description

Added Feature to MachineInventoryLayout to reduce the need for magic numbers when getting ItemStacks (the SingleSlotAccess can be expanded to add more methods for interacting with the inventory)

Todo

  • Move all calls on MachineInventory with magic number access to the SlotAccess System

Checklist:

  • My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you)
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
# Description Added Feature to MachineInventoryLayout to reduce the need for magic numbers when getting ItemStacks (the SingleSlotAccess can be expanded to add more methods for interacting with the inventory) # Todo - [x] Move all calls on MachineInventory with magic number access to the SlotAccess System # Checklist: - [x] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you) - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [x] I have made corresponding changes to the documentation
Rover656 (Migrated from github.com) approved these changes 2022-12-29 20:25:39 +00:00
EpicSquid (Migrated from github.com) approved these changes 2023-01-30 04:08:58 +00:00
EpicSquid (Migrated from github.com) left a comment

Looks great to me, thought I had already responded to this but apparently not. Ship it

Looks great to me, thought I had already responded to this but apparently not. Ship it
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
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: Team-EnderIO/EnderIO#101
No description provided.