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.

Sign up for GitHub

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

Jump to bottom

Migrate inventory.ts to setInventory/getInventory #1354

Open
Pospelove opened this issue Apr 3, 2023 · 0 comments
Open

Migrate inventory.ts to setInventory/getInventory #1354

Pospelove opened this issue Apr 3, 2023 · 0 comments
Assignees
@Pospelove
Labels
area:client

Comments

@Pospelove
Copy link
Contributor

#1353 fixes one issue with the old inventory system

however there is at least one more: when I add 1000000 gold, I end up split into 999999 + 1 inventory entries

we also will be able to deprecate TESModPlatform.pushWornState and TESModPlatform.addItemEx which are... well bad and proven to be crashing in some cases.

We could enable parallel container management after this task.

On setInventory, we work in this PR #1336
There is no getInventory PR at the moment of this issue creation
@Pospelove Pospelove self-assigned this Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Pospelove Pospelove

Labels
area:client
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Pospelove