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

game_store module #1027

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

game_store module #1027

wants to merge 5 commits into from

Conversation

metabobb
Copy link

@metabobb metabobb commented Apr 15, 2019

Changes to game_market:

  • game_market ui/ux improvements
  • added server opcode support 238
  • added client opcode support 233

Changes to compiling on linux:

  • fixes compile errors on linux (kernel: 4.15.0-47-generic). Tested on Ubuntu 18.04

New game module:

  • game_store module

@metabobb
Copy link
Author

metabobb commented Apr 15, 2019

Note Travis build fails with g++: error: unrecognized command line option ‘-no-pie’. Could be due to compiler version mismatch. Travis uses gcc (Ubuntu 4.8.4-2ubuntu1~14.04.3) 4.8.4 while I tested on gcc (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0.

Edit: tested again on gcc version 7.3.0 (Ubuntu 7.3.0-27ubuntu1~18.04) and build completes with no errors.

@hrsantiago
Copy link
Collaborator

@metabobb Most of your code seems fine, however there are major code style issues.
Also, there's a lot of different stuff on a single PR.

@metabobb
Copy link
Author

metabobb commented Apr 16, 2019

@hrsantiago Review the code if you want and I'll make the necessary changes. You can also create a pull request on my repo to fix whatever issues there might be.

Also, sorry that it's so much stuff. With regards to game_market I have added a minor UI/UX change and made the client update market data automatically wherever possible (as far as bot protection allowed). Other commits seem self-explanatory to me.

I've made a thread on OTLand showcasing the module and explaining how to test: https://otland.net/threads/release-game_store-module.264142/

@dudantas
Copy link

Thanks @metabobb it solve my problem, this: https://otland.net/threads/otc-market-bug-and-mounts-bug.264163/#post-2552817

Thank you very much ... I tried to fix this mistake for days, and apparently I would never get it, since I was looking in a totally wrong direction. Its modifications in the market and protocol have totally solved.
Now the market opens without any problems. I will test the other things and give my report, for now I have only to thank and I hope that more people like you appear in the Tibian community.

@metabobb
Copy link
Author

@eduardodantas95 Yes, I have fixed that and also added support for two missing opcodes for client & server. I have tested the market with these changes and it run normally without any issues.

dudantas added a commit to CanaryArchived/client that referenced this pull request May 30, 2020
game_market ui/ux improvements
added server opcode support 238

from edubart#1027
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.

3 participants