Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commitba14acf

Browse files
fix(site): fix disappearing preset selector when switching task template (#20514) (#20564)
Ensure we set `selectedPresetId` to `undefined` when we change`selectedTemplateId` to ensure we don't end up breaking the `<Select>`component by giving it an invalid preset id.---Cherry picked from9629d87 (#20514)
1 parentd0a2e6d commitba14acf

File tree

2 files changed

+126
-1
lines changed

2 files changed

+126
-1
lines changed

‎site/src/modules/tasks/TaskPrompt/TaskPrompt.stories.tsx‎

Lines changed: 120 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -480,3 +480,123 @@ export const CheckExternalAuthOnChangingVersions: Story = {
480480
});
481481
},
482482
};
483+
484+
exportconstCheckPresetsWhenChangingTemplate:Story={
485+
args:{
486+
templates:[
487+
{
488+
...MockTemplate,
489+
id:"claude-code",
490+
name:"claude-code",
491+
display_name:"Claude Code",
492+
active_version_id:"claude-code-version",
493+
},
494+
{
495+
...MockTemplate,
496+
id:"codex",
497+
name:"codex",
498+
display_name:"Codex",
499+
active_version_id:"codex-version",
500+
},
501+
],
502+
},
503+
beforeEach:()=>{
504+
spyOn(API,"getTemplateVersionPresets").mockImplementation((versionId)=>{
505+
if(versionId==="claude-code-version"){
506+
returnPromise.resolve([
507+
{
508+
...MockPresets[0],
509+
ID:"claude-code-preset-1",
510+
Name:"Claude Code Dev",
511+
},
512+
]);
513+
}
514+
if(versionId==="codex-version"){
515+
returnPromise.resolve([
516+
{
517+
...MockPresets[0],
518+
ID:"codex-preset-1",
519+
Name:"Codex Dev",
520+
},
521+
]);
522+
}
523+
returnPromise.resolve([]);
524+
});
525+
spyOn(API,"getTemplateVersions").mockImplementation((templateId)=>{
526+
if(templateId==="claude-code"){
527+
returnPromise.resolve([
528+
{
529+
...MockTemplateVersion,
530+
id:"claude-code-version",
531+
name:"claude-code-version",
532+
},
533+
]);
534+
}
535+
if(templateId==="codex"){
536+
returnPromise.resolve([
537+
{
538+
...MockTemplateVersion,
539+
id:"codex-version",
540+
name:"codex-version",
541+
},
542+
]);
543+
}
544+
returnPromise.resolve([]);
545+
});
546+
},
547+
play:async({ canvasElement, step})=>{
548+
constcanvas=within(canvasElement);
549+
constbody=within(canvasElement.ownerDocument.body);
550+
551+
awaitstep("Presets are initially present",async()=>{
552+
constpresetSelect=awaitcanvas.findByLabelText(/preset/i);
553+
awaituserEvent.click(presetSelect);
554+
555+
constoptions=awaitbody.findAllByRole("option");
556+
expect(options).toHaveLength(1);
557+
expect(options[0]).toContainHTML("Claude Code Dev");
558+
559+
awaituserEvent.click(options[0]);
560+
});
561+
562+
awaitstep("Switch template",async()=>{
563+
consttemplateSelect=awaitcanvas.findByLabelText(/selecttemplate/i);
564+
awaituserEvent.click(templateSelect);
565+
566+
constcodexTemplateOption=awaitbody.findByRole("option",{
567+
name:/codex/i,
568+
});
569+
awaituserEvent.click(codexTemplateOption);
570+
});
571+
572+
awaitstep("Presets are present in new template",async()=>{
573+
constpresetSelect=awaitcanvas.findByLabelText(/preset/i);
574+
awaituserEvent.click(presetSelect);
575+
576+
constoptions=awaitbody.findAllByRole("option");
577+
expect(options).toHaveLength(1);
578+
expect(options[0]).toContainHTML("Codex Dev");
579+
580+
awaituserEvent.click(options[0]);
581+
});
582+
583+
awaitstep("Switch template back",async()=>{
584+
consttemplateSelect=awaitcanvas.findByLabelText(/selecttemplate/i);
585+
awaituserEvent.click(templateSelect);
586+
587+
constcodexTemplateOption=awaitbody.findByRole("option",{
588+
name:/claudecode/i,
589+
});
590+
awaituserEvent.click(codexTemplateOption);
591+
});
592+
593+
awaitstep("Presets are present in original template",async()=>{
594+
constpresetSelect=awaitcanvas.findByLabelText(/preset/i);
595+
awaituserEvent.click(presetSelect);
596+
597+
constoptions=awaitbody.findAllByRole("option");
598+
expect(options).toHaveLength(1);
599+
expect(options[0]).toContainHTML("Claude Code Dev");
600+
});
601+
},
602+
};

‎site/src/modules/tasks/TaskPrompt/TaskPrompt.tsx‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,12 @@ const CreateTaskForm: FC<CreateTaskFormProps> = ({ templates, onSuccess }) => {
270270
</label>
271271
<Select
272272
name="templateID"
273-
onValueChange={(value)=>setSelectedTemplateId(value)}
273+
onValueChange={(value)=>{
274+
setSelectedTemplateId(value);
275+
if(value!==selectedTemplateId){
276+
setSelectedPresetId(undefined);
277+
}
278+
}}
274279
defaultValue={templates[0].id}
275280
required
276281
>

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp