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

Load content servers using new web api endpoint #262

Merged
merged 1 commit into from Jul 2, 2019

Conversation

Projects
None yet
2 participants
@Dahlgren
Copy link
Contributor

commented Jul 2, 2019

ClientServerList message is no longer sent on login

https://steamapi.xpaw.me/#IContentServerDirectoryService

Not sure if you want to retain the contentServersReady and corresponding event for some kind of backwards compatibility.

@DoctorMcKay

This comment has been minimized.

Copy link
Owner

commented Jul 2, 2019

Yes, the contentServersReady property and event need to be included still. You can just set/emit them in the logon callback.

@Dahlgren Dahlgren force-pushed the Dahlgren:bugfix/content-servers branch from 638cfc3 to 2929f21 Jul 2, 2019

@Dahlgren

This comment has been minimized.

Copy link
Contributor Author

commented Jul 2, 2019

Added logic for contentServersReady to logon and logoff callbacks

Load content servers using new web api endpoint
ClientServerList message is no longer sent on login

@Dahlgren Dahlgren force-pushed the Dahlgren:bugfix/content-servers branch from 2929f21 to 8a068ff Jul 2, 2019

@DoctorMcKay

This comment has been minimized.

Copy link
Owner

commented Jul 2, 2019

Thanks, I'll make some changes before pushing it out for backwards compatibility.

@DoctorMcKay DoctorMcKay merged commit 43956ef into DoctorMcKay:master Jul 2, 2019

1 check passed

WIP Ready for review
Details

@Dahlgren Dahlgren deleted the Dahlgren:bugfix/content-servers branch Jul 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.