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

Add serialization for inventories #18

Open
https123456789 opened this issue Dec 21, 2022 · 2 comments · May be fixed by #9
Open

Add serialization for inventories #18

https123456789 opened this issue Dec 21, 2022 · 2 comments · May be fixed by #9
Assignees
Labels
enhancement New feature or request

Comments

@https123456789
Copy link
Member

https123456789 commented Dec 21, 2022

Add a set of methods to the inventory system API for serialization of a item registry.

Ideas

For serializing:

def serialize(self) -> String

And then to deserialize:

def deserialize(self, source: String) -> Inventory

This is planned to done (see this comment), but I wanted to make an issue to sort of remind me.

@https123456789 https123456789 added the enhancement New feature or request label Dec 21, 2022
@https123456789 https123456789 added this to the Inventory System milestone Dec 21, 2022
@https123456789 https123456789 self-assigned this Dec 21, 2022
@LGgameLAB
Copy link
Member

Me likey. I'm leaving this entirely up to you.

@https123456789 https123456789 linked a pull request Dec 21, 2022 that will close this issue
@https123456789
Copy link
Member Author

https123456789 commented Dec 21, 2022

Okay, I was planning on doing it anyway 😅. Glad you like the idea though.

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 a pull request may close this issue.

2 participants