Skip to content

logs and gmail body format updated#78

Merged
Vamsi-o merged 1 commit into
mainfrom
logs-and-email-format
Apr 18, 2026
Merged

logs and gmail body format updated#78
Vamsi-o merged 1 commit into
mainfrom
logs-and-email-format

Conversation

@TejaBudumuru3

@TejaBudumuru3 TejaBudumuru3 commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features

    • Emails now support HTML formatting with markdown-style syntax (bold, italic, links, code blocks)
    • Expanded execution history view shows detailed node-level execution data with collapsible JSON inspection
  • Bug Fixes

    • Fixed text input fields rendering incorrectly as textarea in workflow configuration
    • Loop executions now properly skip rows with unresolved placeholders or empty values
  • UI/UX Improvements

    • Redesigned execution history interface with status icons and expandable node details
    • Enhanced sidebar-aware layout positioning for improved interface usability

@TejaBudumuru3 TejaBudumuru3 requested a review from Vamsi-o as a code owner April 18, 2026 06:43
@coderabbitai

coderabbitai Bot commented Apr 18, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This pull request adds workflow execution result sorting, refactors execution history UI with expanded node details, corrects form field type handling, adjusts canvas layout, implements loop row filtering with skip tracking, and enhances email body formatting with HTML conversion.

Changes

Cohort / File(s) Summary
Workflow Execution & History
apps/http-backend/src/routes/userRoutes/userRoutes.ts, apps/web/app/components/ExecutionHistoryFooter.tsx
Added startAt descending sort to workflow logs endpoint; refactored execution history UI from single-level table to two-level browsing (workflow executions → node executions with expandable details), added sidebar-aware positioning via MutationObserver, replaced status color/label with icon badges, integrated JsonViewer and NodeExecutionRow components.
Configuration & Layout
apps/web/app/workflows/[id]/components/ConfigModal.tsx, apps/web/app/workflows/[id]/page.tsx
Changed textarea rendering to only trigger when field.type === "textarea" (not "text"); adjusted canvas control button bottom offset from 1rem to 60px.
Loop Execution Processing
apps/worker/src/engine/executor.ts
Added skipped count and skippedRows metadata tracking; removed legacy data concatenation onto nodeConfig.body; implemented row filtering to skip empty to/subject fields or unresolved placeholder values, with logging and skip counting.
Email Body Formatting
packages/nodes/src/gmail/gmail.service.ts
Added formatBodyToHtml() helper converting markdown-like syntax (code fences, bold, italic, links, bullet lists) to styled HTML; updated sendEmail to send HTML-formatted messages with explicit MIME-Version header.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

Possibly related PRs

Suggested reviewers

  • Vamsi-o

Poem

🐰 A rabbit hops through logs and nodes,
Where histories now expand and unfold,
Rows skip with grace when placeholders stay,
And emails dress up in HTML today!
Two-level flows and sidebars aligned—
Change well-crafted by CodeRabbit's design. 🌟

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title references only two of six modified files and misses major UI refactoring in ExecutionHistoryFooter, workflow canvas adjustments, and loop execution enhancements. Update title to reflect the scope of changes, e.g., 'Add execution logs sorting, UI improvements, and email body formatting' or similar to capture the broader changeset.
Docstring Coverage ⚠️ Warning Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch logs-and-email-format

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.

@Vamsi-o Vamsi-o merged commit f4876fe into main Apr 18, 2026
1 of 2 checks passed
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