Comments (5)
Implementation brief review: since Search Console API results sorted by click count descending, there isn't need to query more rows and hack the ordering which we are currently doing. We can get rid of the current hack, render a clicks column and simply query the top 10 results.
from site-kit-wp.
@adamsilverstein Do you happen to have any background on why page-analytics
exists? It appears to do exactly the same as sc-site-analytics
. Same about index-status
, it doesn't appear to be used at all? Was it at some point, what was this datapoint meant for?
from site-kit-wp.
As you mentioned, page-analytics
and sc-site-analytics
are currently duplicates, I agree we can safely remove one now.
I did a little git spelunking and found that originally these were two separate similar but distinct data points - one for site wide data and one for page URL specific data:
from site-kit-wp.
QA ✅
from site-kit-wp.
This overall LGTM. Will only note that when all clicks are 0 it looks slightly weird, but is technically correct. We can consider an alternative display down the road if users get confused.
from site-kit-wp.
Related Issues (20)
- Flickering issue before AdSense High potential notification appears. HOT 1
- WP Consent API active tick is squashed at narrow tablet viewports. HOT 2
- Improve the layout of the graphic on the Consent Mode Setup CTA Banner at narrow desktop viewports.
- Reduce the font size on the Consent Mode Setup CTA Banner.
- Release 1.123.0
- Remove Ads bullet point from Consent Mode modal when Ads is not connected. HOT 1
- Extend Consent Mode conditions for determining Ads connection status. HOT 1
- Add filter to allow customisation of Consent Mode defaults. HOT 2
- Extract and rewrite Consent Mode JS snippet.
- Extend Consent Mode to handle additional consent types out of the box.
- Prevent Consent Mode Setup CTA Banner from being shown if the user toggles Consent Mode via Settings.
- Avoid enqueuing duplicate `consent` `update` commands on page load.
- Update the description copy in the Consent Mode modal. HOT 1
- Readme raises new validation warnings HOT 1
- Change the `Ads Conversion ID` Field to `Conversion Tracking ID` HOT 3
- Provide mock GA4 report data via the Tester plugin, including audience reports. HOT 1
- "Set up Google Analytics" step not appearing in SK setup flow HOT 4
- Add Placeholder Content to The `Conversion Tracking ID` Field (`AW-`) HOT 2
- No "Update" button for the "Most popular products...." key metric when googlesitekit_post_date dimension is not present HOT 2
- Reword "Existing Tag" messages and other language changes for SAM HOT 5
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from site-kit-wp.