Skip to content

Add 'cookie policy' on kedro docs #5278

Open
rashidakanchwala wants to merge 7 commits intomainfrom
docs/cookie-policy
Open

Add 'cookie policy' on kedro docs #5278
rashidakanchwala wants to merge 7 commits intomainfrom
docs/cookie-policy

Conversation

@rashidakanchwala
Copy link
Contributor

@rashidakanchwala rashidakanchwala commented Dec 16, 2025

Description

Since Kedro docs is a public website and we track page-level analytics on site visits (although this has been broken since the migration to the new docs and is in the process of being fixed), and following discussions with @Joseph-Perkins as well as the research and security review carried out during the Kedro-Builder work, we created and added this Cookie Policy to clearly document how analytics are used on the site.

Development notes

Developer Certificate of Origin

We need all contributions to comply with the Developer Certificate of Origin (DCO). All commits must be signed off by including a Signed-off-by line in the commit message. See our wiki for guidance.

If your PR is blocked due to unsigned commits, then you must follow the instructions under "Rebase the branch" on the GitHub Checks page for your PR. This will retroactively add the sign-off to all unsigned commits and allow the DCO check to pass.

Checklist

  • Read the contributing guidelines
  • Signed off each commit with a Developer Certificate of Origin (DCO)
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added a description of this change in the RELEASE.md file
  • Added tests to cover my changes
  • Checked if this change will affect Kedro-Viz, and if so, communicated that with the Viz team

Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
@rashidakanchwala rashidakanchwala marked this pull request as ready for review December 16, 2025 11:47
@rashidakanchwala rashidakanchwala requested review from Joseph-Perkins and removed request for yetudada December 16, 2025 11:47
Signed-off-by: rashidakanchwala <rashida_kanchwala@mckinsey.com>
Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know too much about what should be in a Cookie Policy, but I think this captures everything it should 👍


Heap may set analytics cookies on the kedro.org domain for this purpose. The data collected is used solely for documentation improvement and usage analysis.

#### Read the Docs analytics
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can add "Read the Docs" to the headings.yml so it's ignored by Vale


#### Heap analytics

We use Heap Analytics to collect aggregated usage information, such as:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Judging from the Heap website (https://www.heap.io/), I think this should just be "Heap" or "Heap Product Analytics"

Suggested change
We use Heap Analytics to collect aggregated usage information, such as:
We use Heap to collect aggregated usage information, such as:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants