Skip to content

fix: onboarding preview#7238

Merged
jobenjada merged 1 commit intomainfrom
fix/onboarding-preview
Feb 11, 2026
Merged

fix: onboarding preview#7238
jobenjada merged 1 commit intomainfrom
fix/onboarding-preview

Conversation

@Dhruwang
Copy link
Copy Markdown
Member

What does this PR do?

Before

image

After
Screenshot 2026-02-11 at 5 47 16 PM

How should this be tested?

Test onboarding preview survey

Checklist

Required

  • Filled out the "How to test" section in this PR
  • Read How we Code at Formbricks
  • Self-reviewed my own code
  • Commented on my code in hard-to-understand bits
  • Ran pnpm build
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues
  • First PR at Formbricks? Please sign the CLA! Without it we wont be able to merge it 🙏

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR
  • Updated the Formbricks Docs if changes were necessary

@Dhruwang Dhruwang requested a review from jobenjada February 11, 2026 12:29
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 11, 2026

Walkthrough

A previously included OpenText element block was removed from Block 1 in the previewSurvey template within the templates configuration file. The deleted element was created by buildOpenTextElement with the id "preview_input_field_element" and carried an isDraft: true flag. This change reduces the total number of draft elements in Block 1 and modifies the sequence of elements presented before the Next button. No other blocks, logic, or exported signatures were affected.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: onboarding preview' is specific to the change made (removing an element from the onboarding preview survey) and follows Conventional Commits specification.
Description check ✅ Passed The description includes before/after screenshots showing the visual impact, testing instructions, and all required checklist items marked as complete, meeting template requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

@jobenjada jobenjada enabled auto-merge February 11, 2026 14:46
Copy link
Copy Markdown
Member

@jobenjada jobenjada left a comment

Choose a reason for hiding this comment

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

thanks!

@jobenjada jobenjada added this pull request to the merge queue Feb 11, 2026
Merged via the queue into main with commit 8ab8adc Feb 11, 2026
17 checks passed
@jobenjada jobenjada deleted the fix/onboarding-preview branch February 11, 2026 15:02
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