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 sell methods and data to markets? #25

Open
Itangalo opened this issue Dec 25, 2021 · 2 comments
Open

Add sell methods and data to markets? #25

Itangalo opened this issue Dec 25, 2021 · 2 comments

Comments

@Itangalo
Copy link
Owner

Data like 'sellable' and resource names suffixed with 'Sell'. And goods.sell(agent), updating the agent's resources.

Agents should not, I think, have agent.goods.whatev, but agent.goodsId. Thins makes for easier creation from spreadsheet, just as with resources.

@Itangalo
Copy link
Owner Author

I am not 100 percent sure this is a good idea to add to markets. Selling something to the market is in many aspects the same as buying something, which there are already methods for. (Getting gold when selling sugar is the same as buying gold and paying with sugar.)

Will give it a bit more thought.

@Itangalo
Copy link
Owner Author

Ok, I have given it some more thought. This is related to #34.

Selling sugar and getting gold is the same thing as buying gold and paying with sugar, but you could also pay with other sellable items.

Also, you could argue that not all sellable goods should be classified as resources. It thus makes sense to have sell methods.

Will give it even more thought, but mostly on how to implement selling methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant