GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||||||||
GeezerVison is showing github.com — feat(site): share org in personal agent model settings by jakehwll · Pull Request #29122 · coder/coder Conversation
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: c5fe6af162
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| isSaving={saveOverride.isPending && saveMatchesOrganization} | ||
| saveContext={ | ||
| saveMatchesOrganization ? saveOverride.variables?.context : undefined |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Track every in-flight override save
When a user saves another row before the first request finishes, this single mutation observer switches variables to the latest call, so isSaving becomes false for the earlier row even though its request remains active. The user can then edit and resubmit that row; if the older request completes last, it can overwrite the newer selection. Preserve pending state per context, as the previous separate mutations did, or disable all rows while the shared mutation has outstanding requests.
AGENTS.md reference: site/AGENTS.md:L11-L11
Useful? React with 👍 / 👎.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| const defaultModelConfig = MockPersonalDefaultChatModel; | ||
| const claudeModelConfig = MockPersonalClaudeChatModel; | ||
| const reasoningModelConfig = MockPersonalReasoningChatModel; | ||
| const modelOptions = MockPersonalModelOptions; |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
not needed
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.