-
Notifications
You must be signed in to change notification settings - Fork 241
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
Item Management #168
Closed
Closed
Item Management #168
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Made changes for my training wand test. Started work on drop item More work on drop. Added sound and mapped out other messages.
…Left off at sending message for container create.
Added linux compatibility to ConfigManager Added linux compatibility to Dat Loading
* Updated example DatFileDirectory with a trailing slash at the end of the value * Added Path.Combine to DatLoader Finished fixing merge Update for merge - WIP Drop Item - getting the motion packet - slow but sure - WIP Drop Item WIP on drop item - adding mising enum Still not doing this right Drop Item - Getting oh so close to making this work Still working on Drop Item pcap looks the same - he is not bending over and the wand does not appear. He did move once or twice, not sure what I changed.
…pear. He did move once or twice, not sure what I changed. Made changes to make Host config uniform with Port config and added logging/output. (ACEmulator#165) ItemManagement - Close but I am missing something
ogmage78
force-pushed
the
itemmanagement
branch
from
March 24, 2017 14:09
0f5575d
to
7b43356
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Not ready to be pulled.
@ctw me - creates a training wand in your backpack
@ctw ground spawns a training wand on the ground
Drag wand to the ground. It does drop the item and takes it out of the backpack. It does register with the landblock. I had the motion working (bending down to drop the wand and standing back up) but something I changed has got that not working.
The wand does not appear in the visible field. - not sure why.
I am way out of date on coding - and I am sure I have done stuff incorrectly. Any feedback on how to code this better or topics I should read up on will be greatly appreciated. I feel like this is really really close to working. Also, I know the hard coding of data will go away once we have the Weenie data in the database. I put in a number of comments and TODO's to follow up on.
The PCAP is unless I am just blind - almost identical to this one I used as a template. Any help will be appreciated.
Here is a link to the live server PCAP https://www.dropbox.com/s/q0l3wi4mhx0n7ce/drop_item.pcap?dl=0