-
Notifications
You must be signed in to change notification settings - Fork 2
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
Documentation: Add a README for each tracker. #33
Comments
@kiyaGu Are you able to provide more information about copy tracking, or at least who is the ultimate stakeholder of this feature? @adgad I have seen that attention time is part of the quality reads definition but how it is calculated is not specified. Do you happen to know 1) If this attention time code is used for this purpose? 2) If a spec for attention time exists? |
@chrisbrownuk would you be able to review what Spoor events and contexts are used to produce the Quality Reads metric? I suspect it is from the context of the |
There are two events that are contributing to the attention time captured against each page view. This then goes on to calculate the quality read flag. The There is also a deprecated |
Thanks @chrisbrownuk for that extra information. We don't currently have scroll depth accounted for in this repo but I have found where it was previously defined. It appears to create 4 markers on the page at 25% intervals and each time one is moved into view it triggers a Thank you for also pointing out the relationship between the two, this is something I was not aware of and I'll ensure it's preserved. Are you aware of any existing documentation which describes how either of these events should be implemented? Even the old implementation we have to reference was copied from somewhere else with no context. |
@i-like-robots Yes, that's correct with regards to the Unfortunately I don't have any documentation regarding the implementation of these events. |
I'm not sure who the ultimate stakeholder is but our current contact is Philippa Payne. |
Trackers without documentation will not be added to the API of this package.
The text was updated successfully, but these errors were encountered: