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

Power Cells and Power Cell Press #370

Merged
merged 18 commits into from
Aug 22, 2016
Merged

Conversation

Xeni91
Copy link
Contributor

@Xeni91 Xeni91 commented Aug 20, 2016

  • added textures for press and cells
  • added xml for press and cells
  • rendering of press does not work properly don't know why
  • Press job is at the moment just a dummy, have to find out why the job breaks the game

Edit: added better markdown thanks to @Mizipzor

*added textures for press and cells
*added xml for press and cells
*rendering of press does not work properly don't know why
*Press job is at the moment just a dummy, have to find out why the job breaks the game
@Xeni91 Xeni91 mentioned this pull request Aug 20, 2016
*added textures for press and cells
*added xml for press and cells
*rendering of press does not work properly don't know why
*Press job is at the moment just a dummy, have to find out why the job breaks the game
Conflicts:
	Assets/Scripts/Controllers/JobSpriteController.cs
Conflicts:
	Assets/StreamingAssets/Localization/no-BM.lang.meta
@Xeni91
Copy link
Contributor Author

Xeni91 commented Aug 20, 2016

just resolved the merge conflict

Edit: Discussion is in #304

Don't know why it was there
@alexanderfast
Copy link
Collaborator

Here's a friendly tip if you're new to markdown, if you leave a space between the asterisk and the first character Github would render it as an actual list. So instead of

*foo
*bar

You get

  • foo
  • bar

* Sprite of press is loaded
* added job for press
* press takes steel and outputs power cells
@Xeni91 Xeni91 changed the title [WIP] Power Cells Power Cells and Power Cell Press Aug 20, 2016
@Xeni91
Copy link
Contributor Author

Xeni91 commented Aug 20, 2016

Finished evrything I wanted I am just clueless why the Power Cell sprite is not loaded the way it should be

Conflicts:
	Assets/StreamingAssets/LUA/Furniture.lua
* added Localisation for en_US
@vogonistic
Copy link
Collaborator

@Xeni91 Can you remove the change to JobSpriteController.cs and fix the indentation in Furniture.xml?

furniture.AddJob(j)
end
else
furniture.ChangeParameter("smelttime", deltaTime)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Xeni91 I think you have a small mistake here.
In the Furniture.xml you defined the Params as

So it should be presstime not smelttime, or am I reading it wrong?

@alexanderfast
Copy link
Collaborator

At least for .cs files we use spaces instead of tabs. But as far as I know we havent explicitly said anything about .lua and .xml. But maybe we should. I just assumed spaces were used there as well.

Conflicts:
	Assets/StreamingAssets/Localization/en_US.lang
@Xeni91
Copy link
Contributor Author

Xeni91 commented Aug 21, 2016

Thanks for the feedback I will get to it shortly ...

* Added the right param in funiture.lua
* fixed the indention from furniture.xml
* Added the right param in funiture.lua
* fixed the indention from furniture.xml
Conflicts:
	Assets/StreamingAssets/Data/Furniture.xml
@alexanderfast
Copy link
Collaborator

Just wanted to point out that tabs and spaces were already mixed in that file, I'm not blaming you specifically. :)

* indention fix for Furniture.xml
@Xeni91
Copy link
Contributor Author

Xeni91 commented Aug 21, 2016

Yeah you are right ... sometimes there are only 2 spaces for indention and sometimes there are 4, but I used Tab instead of spaces that was zu problem

@WilcoDW
Copy link
Contributor

WilcoDW commented Aug 21, 2016

I made another way of generating power: #466
someone will have to decide which one will be the final one

@Grenkin1988
Copy link
Contributor

#506 Made changes in order to enable power system to react on changes to powerValue

=======
>>>>>>> FuelingSystem
WorldController.Instance.world.jobQueue.cbJobCreated += OnJobCreated;
WorldController.Instance.world.jobQueue.cbJobCreated += OnJobCreated;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this file it seems you went the other way around, spaces became tabs. :)

I think your Monodevelop isnt configured correctly. Check the contribution guidelines, I think there are instructions in there.

@vogonistic vogonistic merged commit df7ae02 into TeamPorcupine:master Aug 22, 2016
@vogonistic
Copy link
Collaborator

Merged!

@Xeni91 Xeni91 deleted the FuelingSystem branch August 22, 2016 07:24
@GamerGeeked
Copy link

@vogonistic Why did you merge this AND uranium?

@vogonistic
Copy link
Collaborator

To get the resources in the game. You can open a new PR to make power cells out of Uranium.

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

Successfully merging this pull request may close these issues.

8 participants