search

heiseonline / shariff-backend-php

👮 PHP backend for Shariff. Shariff enables website users to share their favorite content without compromising their privacy.
http://ct.de/-2467514
133 stars 44 forks source link

Google-Counter immer 0? #131

Closed ahoiroman closed 6 years ago

ahoiroman commented 6 years ago

Hallo zusammen,

der Google-Share-Counter steht bei mir immer auf 0, obwohl der Artikel (z.B. hier: https://spqr.wtf/blog/2018/01/19/migrate-from-bower-to-npm-webpack) geteilt wurde.

Die Konfiguration sollte hier eigentlich passen: https://github.com/SPQRInc/pagekit-shariff/blob/master/src/Controller/ShariffApiController.php#L64-L85

richard67 commented 6 years ago

@SPQRInc Soweit ich weiß beinhaltet der Original-Share-Button von Google seit etwa Mitte 2017 keinen Share-Count mehr, damit er schneller geladen wird. Es gibt dazu ein Bulletin von Google. Ich werde den Link hier später posten. Soweit ich Google verstehe heißt das auch, dass die Zähler nicht mehr abgefragt werden können mit dem vom Shariff-Backend verwendeten Key. Ob ich s mit einem eigenen App-Key funktionieren würde weiß ich nicht. Aber wie ich auf zahlreichen Webseiten sehen konnte sind überall, also auch bei Heise.de alle Google-Zähler Null. Du machst also erstmal nichts falsch.

richard67 commented 6 years ago

https://plus.google.com/110610523830483756510/posts/Z1FfzduveUo

richard67 commented 6 years ago

And https://stackoverflow.com/questions/45157191/google-share-counts-are-not-showing

richard67 commented 6 years ago

I think I will later make a pull request to remove the google+ service from Shariff PHP backend.

richard67 commented 6 years ago

@compeak Could you check those links I provided and decide if the google service should be removed from backend? If so, I can make a pull request.

compeak commented 6 years ago

@richard67 I believe the G+ service should be removed since it does not work as expected anymore.

richard67 commented 6 years ago

Ok, I will make a pull request later today or on weekend.

richard67 commented 6 years ago

Pull request is #132

richard67 commented 6 years ago

As the PR is merged, this issue can be closed.