Skip to content

feat: add getItemsByFunction and getItemsByFunctionAtSlot#323

Merged
LordMidas merged 4 commits intodevelopmentfrom
feat-item-getByFunction
Apr 13, 2024
Merged

feat: add getItemsByFunction and getItemsByFunctionAtSlot#323
LordMidas merged 4 commits intodevelopmentfrom
feat-item-getByFunction

Conversation

@LordMidas
Copy link
Copy Markdown
Member

@LordMidas LordMidas commented Oct 20, 2023

To be merged AFTER #316 (port to modern hooks)

@LordMidas LordMidas added this to the 1.3.0 milestone Oct 20, 2023
@LordMidas LordMidas requested review from Enduriel and TaroEld October 20, 2023 12:40
Comment thread msu/hooks/items/item_container.nut Outdated
@TaroEld TaroEld self-assigned this Feb 17, 2024
@LordMidas LordMidas force-pushed the feat-item-getByFunction branch from 9be46f6 to 6b681aa Compare February 27, 2024 12:25
@LordMidas LordMidas changed the base branch from development to feat-port-to-modern-hooks February 27, 2024 12:25
@LordMidas LordMidas requested a review from TaroEld February 27, 2024 12:25
Comment thread msu/hooks/items/item_container.nut Outdated
@LordMidas LordMidas requested a review from TaroEld February 27, 2024 14:34
Copy link
Copy Markdown
Member

@Enduriel Enduriel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Big fan of the new structure

@LordMidas LordMidas force-pushed the feat-port-to-modern-hooks branch from de55d30 to 983c30b Compare April 13, 2024 04:38
Base automatically changed from feat-port-to-modern-hooks to development April 13, 2024 04:39
@LordMidas LordMidas force-pushed the feat-item-getByFunction branch from f3deb20 to e84c794 Compare April 13, 2024 05:03
@LordMidas LordMidas merged commit b3b8ab6 into development Apr 13, 2024
@LordMidas LordMidas deleted the feat-item-getByFunction branch April 13, 2024 05:04
LordMidas added a commit that referenced this pull request May 10, 2024
)"

This reverts commit b3b8ab6.

Because the functionality added by these functions can be achieved natively by using `.filter` on vanilla `getAllItems` and `getAllItemsAtSlot` functions of item_container.
LordMidas added a commit that referenced this pull request May 21, 2024
)"

This reverts commit b3b8ab6.

Because the functionality added by these functions can be achieved natively by using `.filter` on vanilla `getAllItems` and `getAllItemsAtSlot` functions of item_container.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants