search

PlaceDE-Official / zinnsoldat

Bookmarklet to inject the placeDE bot client into your browser.
53 stars 23 forks source link

update url check for old.reddit users #5

Closed nischolas closed 1 year ago

nischolas commented 1 year ago

User, die in den Reddit-Einstellungen das alte Reddit aktiviert haben, müssen manuell auf new.reddit.com gehen, um das Canvas zu sehen. Das Script war nur auf www.reddit.com ausgelegt.

DooMMasteR commented 1 year ago

war ja klar, dass ich nur die hälfte sehe...

nischolas commented 1 year ago

Haha, in der Sekunde gesehen, dass es das Problem schon an anderer Stelle gab.

https://github.com/PlaceDE-Official/zinnsoldat/commit/f89c4b9b55198f1784257db7f99a24583d761818

DooMMasteR commented 1 year ago

sry kaum zeit, aber die logik passt so nicht... glaube ich

if (!(window.location.href.startsWith('https://www.reddit.com/r/place/') || window.location.href.startsWith('https://new.reddit.com/r/place/'))) {

oder halt 

if (!window.location.href.startsWith('https://www.reddit.com/r/place/') && !window.location.href.startsWith('https://new.reddit.com/r/place/')) {

De Morgan lässt grüßen.

DooMMasteR commented 1 year ago

Sieht gut aus, approved :-) :+1: image bei einem Reddit-Old user ;-)

nischolas commented 1 year ago

sry kaum zeit, aber die logik passt so nicht... glaube ich

Eieiei, war schon im Wochenende ;-)