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

[Revert] Placing task rewards in free slots #2450

Closed
wants to merge 1 commit into from

Conversation

hgtw
Copy link
Contributor

@hgtw hgtw commented Sep 26, 2022

This reverts the changes until an alternative method can be found to properly place rewards in inventory since this doesn't auto stack items

Revert "[Tasks] Let task reward find free bag slots (#2431)"

This reverts commit b4e46c1.

Revert "[Tasks] Place task item rewards in free slots (#2300)"

This reverts commit f381453.

This will probably need to use something like AutoPutLootInInventory or something but I'm not familiar with the inventory api and don't want to look into an alternative right now.

This reverts the changes until an alternative method can be found to
properly place rewards in inventory since this doesn't auto stack items

Revert "[Tasks] Let task reward find free bag slots (EQEmu#2431)"

This reverts commit b4e46c1.

Revert "[Tasks] Place task item rewards in free slots (EQEmu#2300)"

This reverts commit f381453.
@Kinglykrab Kinglykrab changed the title Revert placing task rewards in free slots [Revert] Placing task rewards in free slots Sep 26, 2022
@Akkadius
Copy link
Member

Akkadius commented Sep 26, 2022

Would be nice to not step backwards with this one since this is actually a really nice change (thanks @hgtw)

I put up a change here in our branch 2f6f325 #2449

Below is a GIF of reward_id_list 8003|13005 (Iron ration and long bow)

Long bow is getting added every task completion, iron ration is properly stacking

Screencast from 09-26-2022 12_17_41 AM

Copy link
Member

@Akkadius Akkadius left a comment

Choose a reason for hiding this comment

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

Left comment

@hgtw hgtw closed this Sep 28, 2022
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.

None yet

3 participants