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

Logs nachladen ohne Funktion #745

Closed
arbor95 opened this Issue Aug 8, 2018 · 19 comments

Comments

Projects
None yet
3 participants
@arbor95

arbor95 commented Aug 8, 2018

Describe the bug
Logs nachladen geht nicht

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'https://www.geocaching.com/geocache/GC1HJHK_55' or another cache with enough logs
  2. Click on 'Scrollbar'
  3. Scroll down to 'end of page'
  4. Should see publish log

Expected behavior
if you scroll down enough , you should the publish log.

Screenshots
none

Desktop (please complete the following information):

  • OS: [W10
  • Browser [chrome
  • Version [68.0.3440.84 (Offizieller Build) (64-Bit)

Additional context
all other scripts deactivated.

@Ruko2010 Ruko2010 self-assigned this Aug 8, 2018

@Ruko2010 Ruko2010 added the bug label Aug 8, 2018

@Ruko2010 Ruko2010 added this to the v0.9.10 milestone Aug 8, 2018

@2Abendsegler

This comment has been minimized.

Show comment
Hide comment
@2Abendsegler

2Abendsegler Aug 8, 2018

Owner

@arbor95
Thanks for the message!

@Ruko2010
grafik
VIPs werden entsprechend auch nicht mehr aufgebaut. Bei Button "Show all logs" werden zwar die Logs alle aufgebaut, aber es wird nicht alles richtig dargestellt. Zumindest ist das aber ein Workaround.
Das ist ein Prio high, oder? Ich setz es, ändere es bitte wieder ab, falls du es anders beurteilst. Kannst du das übernehmen?

@arbor95
Workaround: Please use the button "Show all logs".

Owner

2Abendsegler commented Aug 8, 2018

@arbor95
Thanks for the message!

@Ruko2010
grafik
VIPs werden entsprechend auch nicht mehr aufgebaut. Bei Button "Show all logs" werden zwar die Logs alle aufgebaut, aber es wird nicht alles richtig dargestellt. Zumindest ist das aber ein Workaround.
Das ist ein Prio high, oder? Ich setz es, ändere es bitte wieder ab, falls du es anders beurteilst. Kannst du das übernehmen?

@arbor95
Workaround: Please use the button "Show all logs".

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 9, 2018

Collaborator

Ja Prio High ist völlig in Ordnung.
Auch das schau ich mir heute Abend an.
Ich denke das es irgendwie die Funktion "endlessScroll" nicht mehr gibt. Eventuell hatte GC da was eingebunden was nun weg ist. Ich schau mal, ob es reicht wenn wir diese jQuery Erweiterung über unser Script mit einbinden.

Collaborator

Ruko2010 commented Aug 9, 2018

Ja Prio High ist völlig in Ordnung.
Auch das schau ich mir heute Abend an.
Ich denke das es irgendwie die Funktion "endlessScroll" nicht mehr gibt. Eventuell hatte GC da was eingebunden was nun weg ist. Ich schau mal, ob es reicht wenn wir diese jQuery Erweiterung über unser Script mit einbinden.

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 9, 2018

Collaborator

Ich habe versucht das "endlessScroll" script manuell hinzuzufügen aber das scheint nicht zu klappen. @2Abendsegler @CachingFoX hat jemand eine Idee wie man die Funktion ersetzen könnte?

Collaborator

Ruko2010 commented Aug 9, 2018

Ich habe versucht das "endlessScroll" script manuell hinzuzufügen aber das scheint nicht zu klappen. @2Abendsegler @CachingFoX hat jemand eine Idee wie man die Funktion ersetzen könnte?

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 10, 2018

Collaborator

Ich glaube ich weis auch, warum es nicht geht. Wenn ich es richtig sehe, dann wird die Funktion endlessScroll nicht bei uns im Script aufgerufen, sondern direkt auf der GC Seite (über unsafeWindow.$(window).endlessScroll). Wenn ich nun also das Endless Scroll Script bei uns über ein @require hinuzufüge, dann ist es ja nur im GClh sichtbar, aber nicht für code, der direkt auf der GC-Seite ausgeführt wird.

Collaborator

Ruko2010 commented Aug 10, 2018

Ich glaube ich weis auch, warum es nicht geht. Wenn ich es richtig sehe, dann wird die Funktion endlessScroll nicht bei uns im Script aufgerufen, sondern direkt auf der GC Seite (über unsafeWindow.$(window).endlessScroll). Wenn ich nun also das Endless Scroll Script bei uns über ein @require hinuzufüge, dann ist es ja nur im GClh sichtbar, aber nicht für code, der direkt auf der GC-Seite ausgeführt wird.

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 10, 2018

Collaborator

Ok, erster Erfolg:
Wenn man die Zeile
unsafeWindow.$(window).endlessScroll
durch
$(document.body).endlessScroll({
ersetzt und das Script manuell einbindet (über @require) dann kommt schonmal kein Fehler mehr, und die VIP Funktion geht wieder.
Er versucht auch die Logs nachzuladen wenn man zum Seitenende Scrollt, aber es passiert nichts (sie werden nicht hinzugefügt).

Collaborator

Ruko2010 commented Aug 10, 2018

Ok, erster Erfolg:
Wenn man die Zeile
unsafeWindow.$(window).endlessScroll
durch
$(document.body).endlessScroll({
ersetzt und das Script manuell einbindet (über @require) dann kommt schonmal kein Fehler mehr, und die VIP Funktion geht wieder.
Er versucht auch die Logs nachzuladen wenn man zum Seitenende Scrollt, aber es passiert nichts (sie werden nicht hinzugefügt).

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 10, 2018

Collaborator

Scheint doch nicht richtig zu gehen...ich sehe gerade, dass es den gleichen Effekt hat, wenn man den ganzen Block einfach auskommentiert...^^
Wahrscheinlich hat GC seine Funktion zum Nachladen der Logs umgestellt. Wenn jemand gern mitmachen möchte, sei er herzlich willkommen 😉

Collaborator

Ruko2010 commented Aug 10, 2018

Scheint doch nicht richtig zu gehen...ich sehe gerade, dass es den gleichen Effekt hat, wenn man den ganzen Block einfach auskommentiert...^^
Wahrscheinlich hat GC seine Funktion zum Nachladen der Logs umgestellt. Wenn jemand gern mitmachen möchte, sei er herzlich willkommen 😉

@2Abendsegler

This comment has been minimized.

Show comment
Hide comment
@2Abendsegler

2Abendsegler Aug 10, 2018

Owner
Owner

2Abendsegler commented Aug 10, 2018

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 10, 2018

Collaborator

Ich hab jetzt das EndlessScroll Script über Inject auf die GC Seite mit eingebracht, aber da ist immer noch was quer :/
Ich schau mir das heute Abend weiter an. Aktueller Stand ist bei mir im Repo.

Collaborator

Ruko2010 commented Aug 10, 2018

Ich hab jetzt das EndlessScroll Script über Inject auf die GC Seite mit eingebracht, aber da ist immer noch was quer :/
Ich schau mir das heute Abend weiter an. Aktueller Stand ist bei mir im Repo.

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 10, 2018

Collaborator

Ich weis noch nicht genau wann ich heute Abend zu Hause bin, aber ich würde mich dann gleich nochmal damit beschäftigen. Ansonsten hätte ich noch einen Hotfix:

  • das EndlessScroll raus nehmen (logs werden dann aber nicht mehr nachgeladen beim Scrollen nach unten 😔)
  • das hier dafür einbauen (sehr unschön, aber das Script würde erstmal keine Fehler mehr werfen und man kann trotzdem alle Logs anzeigen lassen):

image

@2Abendsegler wenn du damit einverstanden bist bring ich heute Abend eine neue Version raus. Entweder mit dem richtigen Fix (wenn ich es bis dahin hin bekomme), wenn nicht, mit dem (hässlichen) Hotfix. Aktueller Stand mit Hotfix ist in meinem Repo: https://github.com/Ruko2010/GClh/tree/bug/issue-745

Collaborator

Ruko2010 commented Aug 10, 2018

Ich weis noch nicht genau wann ich heute Abend zu Hause bin, aber ich würde mich dann gleich nochmal damit beschäftigen. Ansonsten hätte ich noch einen Hotfix:

  • das EndlessScroll raus nehmen (logs werden dann aber nicht mehr nachgeladen beim Scrollen nach unten 😔)
  • das hier dafür einbauen (sehr unschön, aber das Script würde erstmal keine Fehler mehr werfen und man kann trotzdem alle Logs anzeigen lassen):

image

@2Abendsegler wenn du damit einverstanden bist bring ich heute Abend eine neue Version raus. Entweder mit dem richtigen Fix (wenn ich es bis dahin hin bekomme), wenn nicht, mit dem (hässlichen) Hotfix. Aktueller Stand mit Hotfix ist in meinem Repo: https://github.com/Ruko2010/GClh/tree/bug/issue-745

@Ruko2010 Ruko2010 modified the milestones: v0.9.10, v0.9.11 Aug 10, 2018

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 13, 2018

Collaborator

Ich versuche gerade die Funktion des Nachladens komplett neu zu schreiben. Dabei fällt mir auf, dass ziemlich oft sowas im code steht:

if (isTM === false) {
// Do something
}else{
// do something else
}

Ich weis leider nicht, warum hier zwischen TM und nicht TM unterschieden wird. Meine Frage wäre jetzt, @2Abendsegler da wir nur noch TM unterstützen, könnte ich diese Teile doch gleich umbauen, oder? Ich gehe mal davon aus, dass das alter Code ist, der früher benötigt wurde, als noch TM und GM unterstützt wurden. Wenn ihr nichts dagegen habt, dann schmeiß ich dass raus. Das macht erstens den Code einfacher, und zweitens schaff ich es vielleicht so die Nachladefunktion wieder herzustellen 😉

Collaborator

Ruko2010 commented Aug 13, 2018

Ich versuche gerade die Funktion des Nachladens komplett neu zu schreiben. Dabei fällt mir auf, dass ziemlich oft sowas im code steht:

if (isTM === false) {
// Do something
}else{
// do something else
}

Ich weis leider nicht, warum hier zwischen TM und nicht TM unterschieden wird. Meine Frage wäre jetzt, @2Abendsegler da wir nur noch TM unterstützen, könnte ich diese Teile doch gleich umbauen, oder? Ich gehe mal davon aus, dass das alter Code ist, der früher benötigt wurde, als noch TM und GM unterstützt wurden. Wenn ihr nichts dagegen habt, dann schmeiß ich dass raus. Das macht erstens den Code einfacher, und zweitens schaff ich es vielleicht so die Nachladefunktion wieder herzustellen 😉

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 13, 2018

Collaborator

Ich habe jetzt die Komplette Funktion umgebaut und selbst ein "endless Scroll" implementiert. Der aktuelle Stand ist in meinem Repo: https://github.com/Ruko2010/GClh/tree/bug/issue-745

Das einzige Problem, dass ich noch habe ist, dass GC beim "runterscrollen" ja auch versucht die Logs nachzuladen. Ich schau mal, ob ich das noch abschalten kann, denn sonst kommt immer der Loader, obwohl man die Logs von GC ja nicht mehr braucht.

Ich muss das Script auch noch in den verschiedenen Browsern testen. @2Abendsegler @CachingFoX falls ihr Zeit/Lust habt, könnt ihr es ja auch bei euch mal ausprobieren.

Collaborator

Ruko2010 commented Aug 13, 2018

Ich habe jetzt die Komplette Funktion umgebaut und selbst ein "endless Scroll" implementiert. Der aktuelle Stand ist in meinem Repo: https://github.com/Ruko2010/GClh/tree/bug/issue-745

Das einzige Problem, dass ich noch habe ist, dass GC beim "runterscrollen" ja auch versucht die Logs nachzuladen. Ich schau mal, ob ich das noch abschalten kann, denn sonst kommt immer der Loader, obwohl man die Logs von GC ja nicht mehr braucht.

Ich muss das Script auch noch in den verschiedenen Browsern testen. @2Abendsegler @CachingFoX falls ihr Zeit/Lust habt, könnt ihr es ja auch bei euch mal ausprobieren.

@2Abendsegler

This comment has been minimized.

Show comment
Hide comment
@2Abendsegler

2Abendsegler Aug 14, 2018

Owner
Owner

2Abendsegler commented Aug 14, 2018

@2Abendsegler

This comment has been minimized.

Show comment
Hide comment
@2Abendsegler

2Abendsegler Aug 15, 2018

Owner

Hab mir deine Änderungen im FF angesehen, scheint zu passen mit dem Scrollen.

Der Button "show all logs" bringt allerdings einen Fehler. Bin nicht sicher ob es mit deinen Änderungen und dem HotFix zu tun hat, ist aber meines Erachtens naheliegend. Wenn du den Button drückst und anschließend nochmal drückst gibts auf der Konsole folgende Fehlermeldung:
grafik
Kannst du dir das noch ansehen?

Owner

2Abendsegler commented Aug 15, 2018

Hab mir deine Änderungen im FF angesehen, scheint zu passen mit dem Scrollen.

Der Button "show all logs" bringt allerdings einen Fehler. Bin nicht sicher ob es mit deinen Änderungen und dem HotFix zu tun hat, ist aber meines Erachtens naheliegend. Wenn du den Button drückst und anschließend nochmal drückst gibts auf der Konsole folgende Fehlermeldung:
grafik
Kannst du dir das noch ansehen?

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 15, 2018

Collaborator

Hmm...sieht komisch aus...ja ich schau rein. Danke fürs drüber schauen.

Collaborator

Ruko2010 commented Aug 15, 2018

Hmm...sieht komisch aus...ja ich schau rein. Danke fürs drüber schauen.

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 15, 2018

Collaborator

Ich hab mal mein Script auf Version 0.9.9 zurück gesetzt, da war der Fehler auch schon. Den Button sollte man also nur einmal drücken (dürfen) 😉. Das einfachste wäre glaube ich, ihn deaktiviert zu lassen, wenn der Reload der Logs fertig ist, und ihn nicht wieder zu aktivieren. Dann kann der User ihn nicht nochmal drücken.

Collaborator

Ruko2010 commented Aug 15, 2018

Ich hab mal mein Script auf Version 0.9.9 zurück gesetzt, da war der Fehler auch schon. Den Button sollte man also nur einmal drücken (dürfen) 😉. Das einfachste wäre glaube ich, ihn deaktiviert zu lassen, wenn der Reload der Logs fertig ist, und ihn nicht wieder zu aktivieren. Dann kann der User ihn nicht nochmal drücken.

@2Abendsegler

This comment has been minimized.

Show comment
Hide comment
@2Abendsegler

2Abendsegler Aug 15, 2018

Owner

Das wär aber nicht richtig. Der Button soll mehrfach nutzbar sein, war er ja auch mal. Damit ich zwischen verschiedenen Selektionen wechseln kann. Also beispielsweise die Logs der VIPs, der founds, alle, der not found, alle ... :)
Wir können ja auch ein neues Ticket aufmachen.

Owner

2Abendsegler commented Aug 15, 2018

Das wär aber nicht richtig. Der Button soll mehrfach nutzbar sein, war er ja auch mal. Damit ich zwischen verschiedenen Selektionen wechseln kann. Also beispielsweise die Logs der VIPs, der founds, alle, der not found, alle ... :)
Wir können ja auch ein neues Ticket aufmachen.

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 15, 2018

Collaborator

Ah ok, ich dachte der lädt immer alle Logs. Ja dann mach am besten ein neues Ticket auf.

Collaborator

Ruko2010 commented Aug 15, 2018

Ah ok, ich dachte der lädt immer alle Logs. Ja dann mach am besten ein neues Ticket auf.

@2Abendsegler

This comment has been minimized.

Show comment
Hide comment
@2Abendsegler

2Abendsegler Aug 15, 2018

Owner

Der Button läd ja auch immer alle Logs. Aber ich kann nebendran auf die VIPs drücken um nur die zu sehen, dann wieder auf den Button um alle zu sehen ...
Ich mach mal ein Ticket auf.

Owner

2Abendsegler commented Aug 15, 2018

Der Button läd ja auch immer alle Logs. Aber ich kann nebendran auf die VIPs drücken um nur die zu sehen, dann wieder auf den Button um alle zu sehen ...
Ich mach mal ein Ticket auf.

@Ruko2010

This comment has been minimized.

Show comment
Hide comment
@Ruko2010

Ruko2010 Aug 15, 2018

Collaborator

Ich glaube da ist noch etwas mehr im Argen. Auch der Wechsel zwischen den Sektions geht nicht mehr richtig. Nach dem ersten Wechsel kommt dann ein Fehler.

Collaborator

Ruko2010 commented Aug 15, 2018

Ich glaube da ist noch etwas mehr im Argen. Auch der Wechsel zwischen den Sektions geht nicht mehr richtig. Nach dem ersten Wechsel kommt dann ein Fehler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment