Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.

Change TradeSystem to Trade #4

Closed
Philipp15b opened this issue Sep 4, 2012 · 2 comments
Closed

Change TradeSystem to Trade #4

Philipp15b opened this issue Sep 4, 2012 · 2 comments

Comments

@Philipp15b
Copy link
Contributor

It would be much easier to read and maintain if a Trade was completely seperated from the "TradeSystem", so that you would have a Trade that you can interact with and a TradeSystem that handles all those events.

@geel9
Copy link
Contributor

geel9 commented Sep 4, 2012

Feel free to add me on Steam, "geel9". Or make the changes yourself and
submit a pull request :V

On Tue, Sep 4, 2012 at 12:31 PM, Philipp Schröer
notifications@github.comwrote:

It would be much easier to read and maintain if a Trade was completely
seperated from the "TradeSystem", so that you would have a Trade that you
can interact with and a TradeSystem that handles all those events.


Reply to this email directly or view it on GitHubhttps://github.com//issues/4.

@Philipp15b
Copy link
Contributor Author

Closed with #5.

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants