-
Notifications
You must be signed in to change notification settings - Fork 26
Fix Shop_GiveClientItem (probably unfinished, need tests) #112
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
base: master
Are you sure you want to change the base?
Conversation
Bump version
Fixed for the latest version of SteamWorks
PROBABLY unfinished Fixes Shop_GiveClientItem by handling third param named "count"
Revert one change in a code
TiBarification
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which version of Steamworks you're using? Your changes fails by CI/CD not only on SM 1.11 build (which is allowed to fail), but also on SM 1.10 build.
https://app.travis-ci.com/github/FD-Forks/Shop-Core/jobs/553970168
|
|
||
| /* Stats pusher */ | ||
| public int SteamWorks_SteamServersConnected() | ||
| public void SteamWorks_SteamServersConnected() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need to change it
| public void SteamWorks_SteamServersConnected() | ||
| { | ||
| int iIp[4]; | ||
| char iIp[4]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here, why you have changed it
|
Why you're using this fork instead of original repository? Also i see this fork is marked as "read-only", I don't think it have any support. For building we're use http://users.alliedmods.net/~kyles/builds/SteamWorks/ builds and |
|
@RusJJ please take a look into https://github.com/FD-Forks/Shop-Core/blob/master/.travis.yml file, there are instructions for compilation shop core. |
No description provided.