Skip to content

Conversation

@Harshit-Soni05
Copy link
Contributor

Description

SPRW-2607
Scenario 1: JavaScript editor
Scenario 2: Snippet dropdown
Scenario 3: Script errors are shown inline
SPRW-2606
Scenario 1: Display JavaScript editor in script tabs
Scenario 2: Show “Read Documentation” on the right side
Scenario 3: Snippet dropdown with search on the left bottom
Scenario 4: AI prompt bar with generate button on right bottom

Add Issue Number

Fixes SPRW-2606 , SPRW-2607

Add Screenshots/GIFs

If applicable, add a relevant screenshot/GIF here.

Add Known Issue

If applicable, add any known issues.

Contribution Checklist:

  • The pull request only addresses one issue or adds one feature.
  • I have linked an issue to the pull request.
  • I have linked a PR type label to the pull request.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or GIFs to help explain the change if applicable.
  • I have read the contribution guidelines.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

if (onGeneratePreScript) {
const result = await onGeneratePreScript(testCasePrompt);
if (result?.error) {
isError = true;
Copy link
Contributor

Choose a reason for hiding this comment

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

@Harshit-Soni05, The result may or may not contain an error. So first, I need to check if (generatedContent) — if it exists, you can add it; otherwise, everything else should be treated as an error.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

so do i put simply this here?
if (result?.generatedContent) {
} else {
isError = true;
errorMessage = "Failed to generate script. Please try again.";
}

export let onTestsChange;
export let tabSplitDirection = "horizontal";
export let onGenerateTestCases = undefined;
export let onGeneratePreScript = undefined;
Copy link
Contributor

Choose a reason for hiding this comment

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

add types value for this variables.

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