Skip to content

Change requests initial documentation for beta#699

Open
laliconfigcat wants to merge 21 commits into
masterfrom
changerequests
Open

Change requests initial documentation for beta#699
laliconfigcat wants to merge 21 commits into
masterfrom
changerequests

Conversation

@laliconfigcat

Copy link
Copy Markdown
Member

Describe the purpose of your pull request

  • Provide a clear and concise description of the changes.

Related issues (only if applicable)

  • Provide links to issues relating to this pull request.

How to test? (only if applicable)

  • What part of the application was affected by the changes? What should be tested?

Requirement checklist

  • I have validated my changes on a test/local environment.
  • I have tested that the code snippets I added work. (Leave unchecked if there are no new code snippets.)
  • I have added my changes to the V1 and V2 documentations.

@laliconfigcat laliconfigcat requested a review from a team as a code owner July 3, 2026 12:14
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 3, 2026

Copy link
Copy Markdown

Deploying configcat-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 529b27c
Status: ✅  Deploy successful!
Preview URL: https://910f07f4.docs-xmy.pages.dev
Branch Preview URL: https://changerequests.docs-xmy.pages.dev

View logs

@adams85 adams85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Most of these findings are suggestions. I request changes only where the PR goes against our UX guidelines or doesn't use the exact texts of the UI.

{ url: 'organization', title: 'Organization & Roles' },
{ url: 'targeting/targeting-overview', title: 'Targeting' },
{ url: 'advanced/predefined-variations', title: 'Predefined Variations vs Free‑Form Values' },
{ url: 'change-requests', title: 'Approval flow & Scheduled Changes (Beta)' },

@adams85 adams85 Jul 7, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure this feature qualifies as a basic/fundamental feature.

I suggest discussing this with the UX team.

Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx Outdated
Comment thread website/docs/change-requests.mdx
Comment thread website/docs/change-requests.mdx Outdated

@adams85 adams85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Most of these findings are suggestions. I request changes only where the PR goes against our UX guidelines or doesn't use the exact texts of the UI.

laliconfigcat and others added 2 commits July 8, 2026 12:44
Co-authored-by: adams85 <31276480+adams85@users.noreply.github.com>
adams85
adams85 previously approved these changes Jul 8, 2026
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