Skip to content

Code formatting updates across web app#3531

Merged
steven-tey merged 1 commit intomainfrom
format
Mar 5, 2026
Merged

Code formatting updates across web app#3531
steven-tey merged 1 commit intomainfrom
format

Conversation

@devkiran
Copy link
Collaborator

@devkiran devkiran commented Mar 5, 2026

Summary by CodeRabbit

  • Style
    • Code formatting and whitespace improvements across multiple UI components, animations, and utilities for better consistency and readability.

@vercel
Copy link
Contributor

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dub Ready Ready Preview Mar 5, 2026 5:26am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 95b6335e-ee69-49d4-a076-848862305b02

📥 Commits

Reviewing files that changed from the base of the PR and between bedbd3c and 668a2ce.

📒 Files selected for processing (17)
  • apps/web/app/(ee)/partners.dub.co/(dashboard)/profile/profile-details-form.tsx
  • apps/web/app/api/ai/sync-embeddings/fetch-plausible-pageviews.ts
  • apps/web/app/app.dub.co/(dashboard)/[slug]/(ee)/program/bounties/add-edit-bounty/bounty-criteria-manual-submission.tsx
  • apps/web/app/app.dub.co/(dashboard)/[slug]/(ee)/program/groups/groups-table.tsx
  • apps/web/ui/modals/confirm-set-default-group-modal.tsx
  • apps/web/ui/partners/bounties/claim-bounty-modal.tsx
  • apps/web/ui/partners/use-country-change-warning-modal.tsx
  • apps/web/ui/support/chat-bubble.tsx
  • apps/web/ui/support/chat-interface.tsx
  • apps/web/ui/support/code-block.tsx
  • apps/web/ui/support/message.tsx
  • apps/web/ui/support/program-combobox.tsx
  • apps/web/ui/support/source-citations.tsx
  • apps/web/ui/support/status-indicator.tsx
  • apps/web/ui/support/ticket-upload.tsx
  • apps/web/ui/support/workspace-combobox.tsx
  • packages/ui/src/carousel/nav-bar.tsx

📝 Walkthrough

Walkthrough

This pull request consolidates formatting and styling adjustments across seventeen web app files. Changes include import reorganization, whitespace normalization, className reordering, catch block syntax cleanup, and animation object restructuring—all preserving existing functionality without behavioral modifications.

Changes

Cohort / File(s) Summary
Partners & Profile Components
apps/web/app/(ee)/partners.dub.co/.../profile-details-form.tsx, apps/web/ui/partners/use-country-change-warning-modal.tsx, apps/web/ui/partners/bounties/claim-bounty-modal.tsx
Import relocation, className reordering, styling consolidation, and variable expression formatting with no functional changes.
API & Data Fetching
apps/web/app/api/ai/sync-embeddings/fetch-plausible-pageviews.ts
Plausible API filter formatting restructured as multi-line nested array without altering filter semantics.
Program & Bounty Management
apps/web/app/app.dub.co/.../bounties/add-edit-bounty/bounty-criteria-manual-submission.tsx, apps/web/app/app.dub.co/.../program/groups/groups-table.tsx, apps/web/ui/modals/confirm-set-default-group-modal.tsx
Animation object restructuring, component inlining, and JSX formatting adjustments without altering behavior.
Support Chat UI Components
apps/web/ui/support/chat-bubble.tsx, apps/web/ui/support/chat-interface.tsx, apps/web/ui/support/code-block.tsx, apps/web/ui/support/message.tsx, apps/web/ui/support/program-combobox.tsx, apps/web/ui/support/source-citations.tsx, apps/web/ui/support/status-indicator.tsx, apps/web/ui/support/ticket-upload.tsx, apps/web/ui/support/workspace-combobox.tsx
Catch block syntax normalization, import reordering, type annotation spacing, className formatting, indentation adjustments, and object literal restructuring across chat-related UI components.
Carousel Components
packages/ui/src/carousel/nav-bar.tsx
Parenthesized numeric cast in autoplay transition duration for explicit type casting order clarity.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

  • PR #3486: Directly relates to the useCountryChangeWarningModal hook import relocation in profile-details-form.tsx, which was introduced in that PR.
  • PR #3504: Modifies the same bounty-criteria-manual-submission.tsx file with animation object restructuring that aligns with motion wrapper changes from that PR.
  • PR #3512: Touches the same claim-bounty-modal.tsx file with formatting adjustments to imageMax/maxFiles and urlMax/maxUrls variables.

Suggested reviewers

  • steven-tey
  • willbyers1

Poem

🐰 Hop through the code with a cleanup so neat,
Imports dance past and catch blocks complete,
Classnames arranged in a prettier way,
A formatting spree to brighten the day!
No bugs were harmed in this formatting spree,
Just tidier code for all eyes to see!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 5.26% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: formatting updates across multiple files in the web application.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 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.

Copy link
Collaborator

@pepeladeira pepeladeira left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@steven-tey steven-tey merged commit 191900e into main Mar 5, 2026
11 checks passed
@steven-tey steven-tey deleted the format branch March 5, 2026 17:04
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.

3 participants