-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: implementation for economy #228
Conversation
assets/skins/DefenceDefault.skin
Outdated
@@ -0,0 +1,29 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should have different name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if we believe this is a good default for ... shop screens ... we should just add the skin to Economy.
module.txt
Outdated
@@ -34,6 +34,10 @@ | |||
"id": "Dialogs", | |||
"minVersion": "1.0.0" | |||
}, | |||
{ | |||
"id": "Economy", | |||
"minVersion": "1.1.0-SNAPSHOT" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As @skaldarnar commented in the associated GD PR: we might want to release Economy
once the PR is merged and bump the version number.
@@ -0,0 +1,219 @@ | |||
{ | |||
"type": "ShopScreen", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if this can live in Economy
as well? Or does it differ from the one we have in GD? If so, in how far?
9953b57
to
a0f6ca9
Compare
…Shadow into introduce-economy
This should fix the multiplayer issues. |
Makes use of the implementation from GooeyDefence and requires Terasology/Economy#22, and needs Terasology/LightAndShadowResources#72 to test the shop screen.