-
Notifications
You must be signed in to change notification settings - Fork 103
userscript: failed to join planet; headless: not working either #152
Comments
Yep, am having no joy with this planet either.
…On Sat., 30 Jun. 2018, 5:25 pm darklich, ***@***.***> wrote:
neither userscript nor headless not working.
userscript says failed to join planet, headless stuck here:
Playing as: xxxxxxxxxxx
Running for: 5m 16s
Current level: 14 (6838755 / 7200000)
Exp since start: 0
Current planet: Glory Days Planet II [10.091%] (id 38)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#152>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AmpFfO0esScKEfFH1I-GnmorwDC18BC1ks5uByfagaJpZM4U94L_>
.
|
Zone 0 is for some reason unjoinable on planet 37/38 and since zone 0 is a hard zone the script gets stuck in a loop trying to join it. |
Just a quick dirty workaround to keep you going until a proper fix is released: diff --git a/headless.js b/headless.js
index 6ca976e..89bb11b 100644
--- a/headless.js
+++ b/headless.js
@@ -473,6 +473,9 @@ const GetBestZone = function GetBestZone(planet) {
for (let idx in planet.zones) {
let zone = planet.zones[idx];
+ if (idx == 0 && zone.capture_progress == 0.0)
+ continue;
+
if (!zone.captured) {
if (zone.boss_active) // boss
return zone; Patched script: headless.js Update: check the zone progress as well, so we don't miss the zone when it gets fixed (thanks @Dontmindme2) |
@mrc0mmand thanks man, but in which doc should we put this on? |
Hm, something's wrong. Apparently, I'm doing an amendment, but mainly it doesn't change anything. |
It's about the bosses, please fix it quick, or we will miss the boss, more XP! |
Thanks all, the suggested workaround is working at this end.
…On Sat., 30 Jun. 2018, 6:56 pm BradyThe, ***@***.***> wrote:
As mentioned above by CyBot you can hotfix it yourself, until a proper fix
is released. It is very easy. All you need is headless.js and (for
convenience sake at least) Notepad++.
[image: notepad _2018-06-30_10-54-27]
<https://user-images.githubusercontent.com/8181555/42123548-3fb4764e-7c54-11e8-865d-cc92aa79e20f.png>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#152 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AmpFfKydW7N3HSFCDQYxLQhtzLM9HU43ks5uBz1EgaJpZM4U94L_>
.
|
but this hotfix ignores the bosses |
Yeah, that's a problem. I think there must be another wise variant: use Idle Daddy app for your android device for now, as all dudes said before, until a proper fix. |
AFAIK, the zone 0 is not a boss zone (at least not right now), it's just unjoinable - there's nobody on the planet and it doesn't have the boss mark: The same applies to all planets according to https://wrya.net/tmp/salien-status/ |
You could also try for 474 Edited: since zone hasn't been defined as planet.zones[idx] at that point...my bad That would at least let you join the boss zone when it opens up (i.e. progress has been made), but that is assuming the rest of the script is ready to handle a boss fight. |
Funny, zone with id 0 returns response with this header: |
Zone 0 is bugged on all new planets, it's impossible to join and incorrectly returns the error for boss zone even though it is not a boss zone. The Also that message is just the devs having some fun. It's definetly no ban and it always returns that error when you try to join a Boss Zone using the normal JoinZone api instead of JoinBossZone. Funnily enough it also returns the Steams EResult 17 which === |
I wonder if Valve made it so boss appears in zone 0 after every other zone is defeated? If so, we'll miss it using fix by @mrc0mmand |
I updated the workaround to check the zone progress as well, so we don't miss it once it gets fixed. The code already contains handlers for boss zones, so hopefully it will work once they appear. |
@mrc0mmand the fix doesn't work for me, |
@mrc0mmand can you manually add it to the hub, just for the plain simplicity? I mean it works well and probably in a proper way. |
@Makaka2 done, check the original comment |
Biggie-thankie, man! |
@SuicidalcookieS did you run |
no is that necessary ? |
Yes, it is necessary, as it updates the |
ok now after running |
33d282b |
This is on live now, but I'm very confused as to what valve did here... |
however it's not on userscript yet. |
neither userscript nor headless not working.
userscript says failed to join planet, headless stuck here:
The text was updated successfully, but these errors were encountered: