Skip to content

Conversation

@benbc
Copy link
Contributor

@benbc benbc commented Jan 26, 2026

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 26, 2026

Deploying opensafely-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a662d68
Status: ✅  Deploy successful!
Preview URL: https://db3fef4a.opensafely-docs.pages.dev
Branch Preview URL: https://benbc-ndoo-description.opensafely-docs.pages.dev

View logs

Copy link
Contributor

@evansd evansd left a comment

Choose a reason for hiding this comment

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

This seems correct to me, modulo the small error below.


### Data access which does _not_ go via ehrQL

There are two sorts of circumstances under which data access in OpenSAFELY does not go via ehrQL, each with different behaviour with respect to type 1 opted-out patients' data.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
There are two sorts of circumstances under which data access in OpenSAFELY does not go via ehrQL, each with different behaviour with respect to type 1 opted-out patients' data.
There are two sorts of circumstances under which data access in OpenSAFELY does not go via ehrQL, each with different behaviour with respect to opted-out patients' data.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh damn. I meant to remove that clause altogether. Well spotted -- thank you.

@benbc benbc marked this pull request as draft January 26, 2026 16:12
@benbc benbc force-pushed the benbc/ndoo-description branch from c10c6d0 to a662d68 Compare January 26, 2026 16:13
@benbc benbc marked this pull request as ready for review January 27, 2026 09:44
@benbc benbc merged commit 7f8d660 into main Jan 27, 2026
2 checks passed
@benbc benbc deleted the benbc/ndoo-description branch January 27, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants