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

Make functions available for non-player inventories (stashes) #539

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

DrMeepso
Copy link

@DrMeepso DrMeepso commented Jun 23, 2024

Description

Makes a few functions like "CanAddItem" and "SetInventory" usable on stashes
Added:
"CreateInventory" - Creates a stash so it can be made ahead of time before using "OpenInventory" for things like adding items
"GetInventory" - Returns the inventory given via the identifier pram, used to things like checking what items are in a stash
"RemoveInventory" - Removes a inventory from the "Inventories" table so it wont get saved to the SQL db

Pretty much just functions useful for making and interfacing with stashes for things like custom jobs

Checklist

  • My code fits the style guidelines.
  • My PR fits the contribution guidelines.
  • I have personally loaded this code into an updated qbcore project and checked all of its functionality.

@DrMeepso DrMeepso changed the title Make inventory available for non-player inventories (stashes) Make functions available for non-player inventories (stashes) Jun 23, 2024
@DrMeepso DrMeepso marked this pull request as draft June 23, 2024 01:50
@DrMeepso
Copy link
Author

DrMeepso commented Jun 23, 2024

Everything has been tested & works!

@DrMeepso DrMeepso marked this pull request as ready for review June 23, 2024 02:06
@GhzGarage GhzGarage self-assigned this Jul 25, 2024
@IamLation
Copy link
Contributor

Great PR, would love to see this

@GhzGarage GhzGarage added the enhancement New feature or request label Sep 5, 2024
@DrMeepso
Copy link
Author

fixed linting issues.

@GhzGarage GhzGarage linked an issue Oct 17, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] AddItem to trunk
3 participants