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

Made Init be able to use appid instead of needing an steam_appid.txt #461

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

Conversation

Redageddon
Copy link

No description provided.

@rlabrecque
Copy link
Owner

This is probably better for something like https://github.com/rlabrecque/Steamworks.NET-SteamManager to handle, since this fundamentally changes how the Steam API would work. This would need a bunch of documentation explaining that steam_appid.txt doesn't work with Steamworks.NET and stuff like that.

@rlabrecque
Copy link
Owner

Actually I kind of changed my mind slightly, since we write the default steam_appid.txt when we launch, one thing that might be even better is to use the Unity SettingsProvider to accomplish this?

@Redageddon
Copy link
Author

I didn't use this with unity. I used it in my own project. I just hated that there was no way to change the appId in the same program that was built in. So I spent a painfully long time trying to figure out how to do it, and found that it uses envvars. So I decided to add it here because it make SOOOOO much more sense to initialize the id though startup itself rather than though an arbitrary file. Especially considering that chancing the ID and shutting down and restarting the SteamAPI, litearlly does not update with the new id in the file. Which is where this pain started with.

@Redageddon
Copy link
Author

could always default, if a steam id isnt passed, it still reads from file, so it doesnt break compatability

@Redageddon
Copy link
Author

I just made this hastily, because It annoyed the crap out of me that there was no build in way to change the appId even when that should seem like a very basic function that should be there.

@Redageddon
Copy link
Author

I fixed the merge code so it doesn't break compatibility.

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.

2 participants