Skip to content

fix(platform): add missing import guidance for customers and vendors#1314

Merged
yannickmonney merged 1 commit into
mainfrom
fix/import-guidance
Apr 10, 2026
Merged

fix(platform): add missing import guidance for customers and vendors#1314
yannickmonney merged 1 commit into
mainfrom
fix/import-guidance

Conversation

@yannickmonney
Copy link
Copy Markdown
Contributor

@yannickmonney yannickmonney commented Apr 10, 2026

Summary

  • Add format hint bullet to vendor import form in both manual entry and file upload tabs
  • Highlight customer import status note in blue to match product import pattern
  • Add formatHint translation key for vendors in en.json and de.json

Closes #1301, closes #1302

Summary by CodeRabbit

  • New Features

    • Vendor import forms now display formatting guidance to help users properly structure bulk import data (email, name, locale).
  • Style

    • Enhanced visual prominence of status notes in customer import form with improved text highlighting.

Add formatHint bullet to vendor import form guidance in both manual
entry and file upload sections. Highlight customer statusNote with
blue text in both sections to match the product import pattern.
Add formatHint translation key for vendors in en.json and de.json.
Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 5b3fcce5-6d2c-4b86-96ff-a92052d8fe0d

📥 Commits

Reviewing files that changed from the base of the PR and between 76038fd and 042ef9f.

📒 Files selected for processing (4)
  • services/platform/app/features/customers/components/customer-import-form.tsx
  • services/platform/app/features/vendors/components/vendor-import-form.tsx
  • services/platform/messages/de.json
  • services/platform/messages/en.json

📝 Walkthrough

Walkthrough

The PR makes targeted improvements to import forms for customers and vendors. It adds visual styling to the status note in the customer import form by applying the text-blue-600 Tailwind class to highlight the text. For vendors, it adds a new format hint list item to both manual and file upload import sections, supported by corresponding English and German localization strings that describe the expected input format (email, optional name and locale, one vendor per line).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~8 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding missing import guidance for both customers and vendors, which is the primary focus of all changes.
Linked Issues check ✅ Passed All code changes directly address requirements from linked issues #1301 and #1302: adding format hints for vendor imports and visually highlighting customer import status notes.
Out of Scope Changes check ✅ Passed All changes are directly related to import guidance requirements: customer form styling, vendor form hint, and necessary translation keys for both customers and vendors.

✏️ 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 fix/import-guidance

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.

@yannickmonney yannickmonney merged commit 38e9dc9 into main Apr 10, 2026
24 checks passed
@yannickmonney yannickmonney deleted the fix/import-guidance branch April 10, 2026 11:22
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.

There is no guidance for importing vendor There is no guidance for importing customer

1 participant