If user not logged in and visiting a cache page directly, he has no chance to sign in

[Ruko2010]

[Ruko2010]

Menue is hidden, so user can't log in to see the cache page

[2Abendsegler]

Das passiert nur bei PMO Caches. Hängt damit zusammen, dass der GClh auch ausgeführt wird selbst wenn kein User angeloggt ist. Das fand ich immer schon blöd, eigentlich sollte der GClh dann gar nicht laufen. Dann wurde aber der Facebook Login angezeigt.

Ich hab eine einfache Lösung. Soll ich die mal in den Collector stellen?

Off-Topic: Falls du keine Werbung auf den GC Seiten haben möchtest. Da gibt es ein winziges Script. :)
https://github.com/2Abendsegler/GC_ad_blocker

[Ruko2010]

Ja du kannst die Lösung gern in den Collector stellen. Ich hab mir schon gedacht, dass es nur bei PMO ist, wollte mir es aber noch genauer anschauen.
Werbung: Die lass ich lieber, immerhin finanziert das Geocaching mit. Und wenn es hilft die Premiumkosten etwas weiter unten zu halten, lass ich die gern anzeigen 😉.

[2Abendsegler]

Hab das "Hide Facebook" weiter nach oben geschoben und steig dann anschließend aus dem maingc aus. Das ist das Einzige was auch ohne User laufen sollte.

Off-Topic: Ich glaube das finanziert nur dann, wenn man draufklickt.

[2Abendsegler]

Fix: If user not logged in and visiting a cache page directly, he has no chance to sign in: [Issue #594]

[Ruko2010]

Danke für den Fix...

Off-Topic: Ich glaube das finanziert nur dann, wenn man draufklickt.

Ich glaube nicht. zumindest ist es bei Google Anzeigen so, dass man schon bezahlt wenn das nur angezeigt wird. Aber du hast recht, es gibt Netzwerke wo man erst was bekommt, wenn man drauf klickt. Aber so es drum. wenn die Werbung nicht überhand nimmt, dann stört sie mich auch nicht.

[2Abendsegler]

Die Änderungen zu diesem Issue sind auf dem Collector verschwunden. Ich schätze mal, das hängt mit deinen letzten Merges zusammen. Ich weiß nicht ob auch andere Dinge verschwunden sind. Kannst du bitte mal prüfen?

Ich stelle einen PR zur Verfügung mit dem die Änderungen zu diesem Issue und dem #601 wieder eingebaut werden können. Bitte spiel du den PR ein.

[Ruko2010]

hast du in der richtigen Branch geschaut? Also wenn ich den aktuellen Stand des collectors anschaue, dann ist dein Commit mit dem Facebook schon (noch?) drin!
Hast du vielleicht in meinem Collector geschaut? Da sind deine Änderungen nicht drin, und das hat seine Richtigkeit.

[Ruko2010]

@2Abendsegler : das ist ein Screenshot aus dem aktuellen Collector

[2Abendsegler]

Jetzt holen wir mal tief Luft und machen alles nochmal. :)
Das ist der Collector bei mir:

Und das ist der neue PR bei dem es mir aufgefallen ist:

Schau mal bitte nochmal. :)

[Ruko2010]

Schau mal bitte in zeile 510 des aktuellen collectors. Da ist diese Facebookfunktion bei mir schon drin (in deinem Repo, im aktuellen Collector (siehe URL im Browser)). Aber das ist natürlich die Stelle von var mainGMaps = function() {. Deine Facebook funktion ist bei var mainGC = function() { drin. Muss die in beide rein?

[Ruko2010]

Dein Commit vor 8 Tagen, hatte es an die Stelle var mainGMaps = function() { eingefügt. Und das ist wie gesagt auch noch drin.

[2Abendsegler]

Das ist ja völlig falsch, was ein Dreck, mein Fehler. Ich erinnere mich, dass ich beim letzten PR schon die gleichen Probleme hatte. Sorry.
Ich korrigier das jetzt.

[Ruko2010]

Gut das wir drüber geredet haben 😃.