docs(ep-v2): name Groq and the model in the Ask AI disclosure - #4397
Closed
Bishibop wants to merge 1 commit into
Closed
docs(ep-v2): name Groq and the model in the Ask AI disclosure#4397Bishibop wants to merge 1 commit into
Bishibop wants to merge 1 commit into
Conversation
Restores what #4391 deliberately withheld while Groq was absent from the Infrastructure and Subprocessors page. Depends on #4396 adding the Groq row. Must not merge before it: naming a processor missing from the published list is the exact inconsistency #4391 avoided, and Vale.Spelling fails on Groq until #4396 adds the vocabulary entry.
✅ Deploy Preview for replicated-docs-upgrade ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for replicated-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
Author
|
Folded into #4396. Keeping the subprocessor row and the disclosure naming in one atomic change so the disclosure can never name a processor that is absent from the published list — with two PRs that invariant depended on merge ordering. Neither half is urgent enough to want them separate; both wait on the same sign-off. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Restores the provider name and model id to the Ask AI privacy disclosure. Final piece of sc-139273.
Warning
Merge order is load-bearing. This must land after #4396, and after #4391.
Vale.Spelling, because docs: add Groq to the subprocessor list and name it in the Ask AI disclosure #4396 owns the vocabulary entry.mainautomatically once docs(ep-v2): add Portal Features page documenting Ask AI #4391 merges.If #4396 is rejected — that is, if Groq is not an approved subprocessor — close this PR rather than merging it. #4391 is correct and complete without it.
What it changes
Three lines, all in
enterprise-portal-v2-portal-features.mdx:openai/gpt-oss-120bmodel as the processor, replacing "a third-party AI provider"No change to the substance of the disclosure. What data is transmitted — the question, up to the last 10 messages, and the entitlement-filtered content — was already documented in #4391 and is untouched here. This PR adds only the identity.
Why it was split out in the first place
#4391 documents Ask AI, which is live and was shipping to customers with no documentation at all. Holding that page until a subprocessor listing landed would have kept a live data flow undocumented for an unbounded period, so the naming was deferred and the page shipped with the disclosure intact.
The model id went with the provider deliberately, and the reason is easy to miss:
openai/gpt-oss-120breads as OpenAI, and OpenAI is already listed as a subprocessor. Naming the model while withholding the provider would have pointed readers at the wrong processor. The two are restored together for the same reason they were removed together.Vale
Depends on
Groqinstyles/config/vocabularies/ThirdPartyProducts/accept.txt, which #4396 adds. Not duplicated here — two PRs adding the same accept-list line would conflict. Verified clean once that entry is present.