Skip to content

Split reviewer toggle and API access#2

Open
andrewsanchez wants to merge 1 commit into
pr4289-split/backend-protobuf-rpcfrom
pr4289-split/reviewer-toggle-api-access
Open

Split reviewer toggle and API access#2
andrewsanchez wants to merge 1 commit into
pr4289-split/backend-protobuf-rpcfrom
pr4289-split/reviewer-toggle-api-access

Conversation

@andrewsanchez
Copy link
Copy Markdown

Source split from ankitects#4289.

Breakdown sections covered:

    1. Preference toggle: new_reviewer opt-in
    1. Main-window API access

Source fidelity:

Follow-up notes from the breakdown:

  • The reviewer toggle requires restart because API access is configured when the webview is initialized.
  • API access should ideally be scoped to the reviewer webview rather than the whole main window.
  • The toggle may be better stored in profile metadata instead of collection config.

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