Skip to content

Focus opt-in guide on setup and link to the SDK example - #29

Open
roncohen wants to merge 5 commits into
mainfrom
fix/opt-in-loading-and-errors
Open

roncohen wants to merge 5 commits into
mainfrom
fix/opt-in-loading-and-errors

Conversation

@roncohen

@roncohen roncohen commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Remove the duplicated React implementation and detailed SDK API/loading guidance from the opt-in guide.
  • Keep the guide focused on flag configuration, required context, user/company scope, and access behavior, with a direct link to the React SDK opt-in example.
  • Update the handbook's description of the guide so it no longer promises an embedded step-by-step implementation.

The React SDK documentation is the single source of truth for the implementation example. No files under sdk/ are changed; they remain managed in the JavaScript repository and copied here.

Related SDK fixes: reflagcom/javascript#729

Validation

  • Validated all 7 local links and heading anchors in the changed pages, including the SDK example link.
  • Confirmed the guide contains no duplicated code examples.
  • git diff --check passed.

Only guides/self-opt-in.md and the handbook's link description are changed. Unrelated local API docs and assets are excluded.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 15, 2026 10:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 15, 2026 10:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@roncohen roncohen changed the title Correct opt-in loading, error handling, and authorization guidance Focus opt-in guide on setup and link to the SDK example Sep 15, 2026
Copilot AI review requested due to automatic review settings September 15, 2026 11:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 16, 2026 09:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

2 participants