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

Create Training Wand - Demo work for item creation. #136

Closed
wants to merge 4 commits into from
Closed

Create Training Wand - Demo work for item creation. #136

wants to merge 4 commits into from

Conversation

ogmage78
Copy link
Contributor

Added some test code to create a training wand. As part of this work, I put in some fixes in sizes, added AnimationFrame. While this is not all that spectacular - it perfectly matches live server pcaps in every field.

I would like to work on persisting this out to the database. I assume we will need to implement a next object_id sequence as part of this. In the test code it just sets an arbitrary object_id for the wand since it is not yet persisted. Any feedback on the code would be appreciated - I am way out of date, but studying hard to get caught up so I can help. Pea gave me the final piece of the puzzle - without being able to set animation frame, setting the flag caused an alignment mismatch. God this was fun. I forgot how much I used to love coding.

Copy link
Contributor

@Mogwai-TheFurry Mogwai-TheFurry left a comment

Choose a reason for hiding this comment

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

This will need heavy refactoring after the landblock PR. Putting this on hold for now.

@ogmage78
Copy link
Contributor Author

Ok - thanks for checking it out. I had to get a toe wet somehow ... ;)

Forbiddenz and others added 3 commits March 16, 2017 14:50
* WorldObject Updates

* changes

* fixes

* GuidManager Start + World Update Changes

* Cleaned up.

* more clean up.

* I got you you little // model id 0x000026 bastard..

* Fixed
@ogmage78
Copy link
Contributor Author

Closing and resubmitting

@ogmage78 ogmage78 closed this Mar 16, 2017
@ogmage78
Copy link
Contributor Author

Closing and resubmitting

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

Successfully merging this pull request may close these issues.

None yet

3 participants