-
Notifications
You must be signed in to change notification settings - Fork 54
Flow builder UI #3689
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: whatsapp-forms-phase-2
Are you sure you want to change the base?
Flow builder UI #3689
Conversation
…using dnd-kit feat: create Configure component for WhatsAppForms with FormBuilder and Preview style: add CSS styles for Configure, FormBuilder, Screen, and Preview components refactor: update WhatsAppForms to include Google Sheet URL and remove unused formJson state fix: add configure action in WhatsAppFormList for navigating to the Configure page
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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. Comment |
…add, update, delete, and reorder functionalities for content items
…gement features, including screen state handling and improved styling
Glific
|
||||||||||||||||||||||||||||
| Project |
Glific
|
| Branch Review |
whatsapp-forms-2/flow-builder-ui
|
| Run status |
|
| Run duration | 27m 51s |
| Commit |
|
| Committer | Akansha Sakhre |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
2
|
|
|
0
|
|
|
0
|
|
|
190
|
| View all changes introduced in this branch ↗︎ | |
…including screen name and field label editing, and improved JSON viewer styling
…review functionality with dynamic content rendering
…revisions and improve navigation
…e management and styling - Enhanced FormBuilder component to accept expanded screen and content IDs as props. - Refactored styles in FormBuilder, Screen, JSONViewer, Preview, and Variables components for consistency. - Updated variable naming conventions in CSS modules to follow PascalCase. - Improved the layout and structure of the Preview component for better user experience. - Removed unused variables and streamlined the rendering logic in the Variables component.
…ality in WhatsApp forms
…ck in WhatsApp forms
…and improve variable display
…ntend into whatsapp-forms-2/flow-builder-ui
Normalize hex color case, remove extra blank lines, and ensure files end with newlines for consistency.
|
@priyanshu6238 Please fix the conflicts |
According to the new form builder, we don’t need to go to the playground, right? So why do we still have this when we create the form?
@priyanshu6238 @shijithkjayan any idea? I might be missing something.
|
Yes, we don’t need this. I reviewed the code in the morning . there are a few details missing in the UI, and it’s not fully ready yet. |
okat let;s remove it then |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## whatsapp-forms-phase-2 #3689 +/- ##
==========================================================
- Coverage 82.75% 79.81% -2.95%
==========================================================
Files 317 334 +17
Lines 11884 12633 +749
Branches 2492 2723 +231
==========================================================
+ Hits 9835 10083 +248
- Misses 1286 1749 +463
- Partials 763 801 +38 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…tsapp-forms-2/flow-builder-ui
The cross button and the three dots don’t do anything. Why do we have them? @priyanshu6238 |



whatsapp-forms/configure/1