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

Update Pet System #939

Merged
merged 10 commits into from Oct 20, 2013
Merged

Update Pet System #939

merged 10 commits into from Oct 20, 2013

Conversation

Baeumchen
Copy link
Contributor

This will fix:

#794
#933
#931
#579
#535

Also it fixes the React States, i.e. Passive, Aggressive and Defensive. Commands are working to. Attack, Follow, Stay. Of course the Spells are working to.

http://www.youtube.com/watch?v=6WVJV38daYk

@Bootz
Copy link
Contributor

Bootz commented Sep 27, 2013

Need one more commit,
Codestyle, white space cleanup
and then can be squashed to one changeset.
.
;) ... I haven't tested yet since moved to windows phonE,
haven't converted my RDC over yet from Android.

@Baeumchen
Copy link
Contributor Author

Sorry, i don't know how to add new commit's, ore how the codestyle is wished :s I only use github to copy + paste my codes online ^^ Don't really know the system :s :D

I'm pretty sure, that this will compile with 0 errors and work like a charm :P

@SkyFire
Copy link
Member

SkyFire commented Sep 28, 2013

If it doesn't. Blame boots ;) Will check this out shortly.

@Baeumchen
Copy link
Contributor Author

Sorry for the commits with the Startup errors i dunno why that happens.. It's the 2nd time now .__. Well those are just a few fixes for startup

i.E. that fixes Stranglethorn Vale Startup error and wrong zone error

@ghost
Copy link

ghost commented Oct 7, 2013

So when will this get merged?

@SkyFire
Copy link
Member

SkyFire commented Oct 8, 2013

When I have the opportunity to test it and review it. This is a very busy time for me right now.

SkyFire added a commit that referenced this pull request Oct 20, 2013
@SkyFire SkyFire merged commit 318df1c into ProjectSkyfire:master Oct 20, 2013
@SkyFire
Copy link
Member

SkyFire commented Oct 20, 2013

Tested ok. will do a cleanup on it.

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.

None yet

3 participants