Skip to content
This repository has been archived by the owner on Feb 14, 2025. It is now read-only.

Ending Resources #96

Merged
merged 26 commits into from
Feb 21, 2017
Merged

Ending Resources #96

merged 26 commits into from
Feb 21, 2017

Conversation

timgrossmann
Copy link
Member

@timgrossmann timgrossmann commented Feb 20, 2017

Please merge #87 First

Part of Issue #82

I'm not really sure if a separate component fits this case, but I thought the health component is better for actual "active" elements like robots/towers.

What do you think?

Edit: That is what i meant:

screen shot 2017-02-20 at 20 27 53

The collectable components now get removed partially until fully removed

Code review is really needed here, right now the renderer is separate and probably could be integrated better

Update:

There are now 3 "types" or resources:

  • red
  • green
  • blue

You can set the values of the gems aka. resources with different "chunks" e.g. RED_GEM with 25/chunk and BLUE_GEM with 10/chunk.
The robots also carry the same color of resource with them instead of just blue

@paulsonnentag
Copy link
Collaborator

Just a quick note for the future. If you create a pull request which is forked from another branch which hasn't been merged can you reference that branch so the pull requests can be easily reviewed in order

@H3rby7 H3rby7 merged commit cc71e17 into develop Feb 21, 2017
paulsonnentag pushed a commit that referenced this pull request Mar 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants