search

GerRudi / SimpleR-RotMG

A convenient way to play Realm of the Mad God
MIT License
4 stars 1 forks source link

Use official permalink to grab version #36

Closed GerRudi closed 6 years ago

GerRudi commented 6 years ago

No need for regex anymore. Deca now provides an always up-to-date link via https://realmofthemadgodhrd.appspot.com/client (As announced with patch X.22.0)

vScherb commented 6 years ago

So the old routes permanently removed or already shut down.

When can this permanent replacement be implemented, otherwise I could take it myself if you expalin it a bit more. Is there a similar solution for the beta program of RotMG?

GerRudi commented 6 years ago

I don't see a reason to keep the old solutions and I think they should be removed completely. Even as a fallback - if the official link doesn't work, the fallback won't work either.

As announced today, there's also a link for the testing server: http://testing.realmofthemadgod.com/client The testing server goes public tomorrow. I can't even confirm that the old implementation is still working (see error in v1.2.0), because it hasn't been available for the public in over two years. To make sure the testing server works, we should update to the official solution asap.

Edit: It does throw the known language error with the current implementation as well as the official one.

GerRudi commented 6 years ago

Devs responded - https://rotmgtesting.appspot.com/client is working, will try to push hotfix soon.

vScherb commented 6 years ago

There are three kinds of starting the game:

  1. Play regularly
  2. Play on testing
  3. Play on production with Kongregate
    • get the current version from the version.txt
    • play with the load on demand client (AGCLoader)

This will be implemented and pushed for the new minor version.

Additional

Removal of the setting "Use AGCLoader instead of AssembleeGameClient" to avoid confusion for the enduser. No more need to store it in the permanent storage of the selected settings.

@GerRudi you see any additions to this or is this correct and can be implemented?

GerRudi commented 6 years ago

Looks good

vScherb commented 6 years ago

Will be closed when it gets merged into master with a new release.

vScherb commented 6 years ago

@GerRudi can this issue be closed now?

41 got merged and was released with version 1.2.3 or does it depend on the referenced issue #45?

GerRudi commented 6 years ago

Fixed in v1.2.3