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

SteamFriends and SteamUser Changes #596

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

Conversation

Crytilis
Copy link
Contributor

@Crytilis Crytilis commented Sep 30, 2021

This PR would implement the following changes as listed on the commits;

  • Added AdvertiseGame to SteamUser so that it may be used for clients.
  • PersonaChange enum is now set to public rather than internal so it may used with the PersonaStateChange callback.
  • Modified the PersonaStateChange callback for SteamFriends to include the PersonaStateChange_t flag as defined in the PersonaChange enum.
  • Removed the F word from the readme in the event that prospective employers are looking at a forked GitHub.

@Crytilis Crytilis changed the title SteamFriend and SteamUser Changes SteamFriends and SteamUser Changes Sep 30, 2021
Changed badge back to the appropriate link.
@Crytilis
Copy link
Contributor Author

Crytilis commented Oct 2, 2021

Must have overlooked that, the correction has been made. I did make a few more changes some of which were for compilation testing.

  • Updated Test Adaptor and Framework packages
  • Updated Newtonsoft packages
  • Modified the folder path to account for .NET6 changes as the compiler was throwing errors regarding path until the change was made.
  • Set the steam_api.json file to copy to output upon compilation to prevent compile errors as it was throwing on it missing until this change was made.

@garrynewman
Copy link
Member

There's a few too many unexplained changes here for me to pull this.

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.

None yet

2 participants