Updated the Flatfile importer#421
Merged
sriramveeraghanta merged 2 commits intomasterfrom Mar 27, 2026
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughDocumentation updates across three areas: CSV importer guidance now includes UTF-8 encoding instructions with an updated screenshot reference; Flatfile importer docs expanded with a complete CSV-to-Plane workflow, field mapping rules, and import management details; billing docs clarify free-trial-to-Free-plan transition and billing behavior. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
docs/workspaces-and-users/billing-and-plans.md (1)
153-153: Consider clarifying the billing behavior for all trial scenarios.The sentence structure leaves ambiguous what happens if a payment method was added during the trial but no upgrade occurred. The conditional phrase "if a payment method was never added" implies there might be a different outcome when one was added, but that scenario isn't addressed.
Consider restructuring for clarity:
📝 Suggested improvement for clarity
Option 1 - Address both scenarios explicitly:
-After 14 days, your workspace automatically reverts to the Free plan if you haven't upgraded to a paid subscription, and you won't be charged if a payment method was never added. +After 14 days, your workspace automatically reverts to the Free plan if you haven't upgraded to a paid subscription. No charges are applied since the trial doesn't require a payment method.Option 2 - Split into separate sentences:
-After 14 days, your workspace automatically reverts to the Free plan if you haven't upgraded to a paid subscription, and you won't be charged if a payment method was never added. +After 14 days, your workspace automatically reverts to the Free plan if you haven't upgraded to a paid subscription. Since no payment method is required for the trial, you won't be charged.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/workspaces-and-users/billing-and-plans.md` at line 153, The sentence "After 14 days, your workspace automatically reverts to the Free plan if you haven't upgraded to a paid subscription, and you won’t be charged if a payment method was never added." is ambiguous about the case where a payment method was added during the trial but no upgrade occurred; rewrite the line (the sentence starting with "After 14 days...") to explicitly cover both outcomes—e.g., state that the workspace reverts to Free after 14 days and that users will not be charged unless they actively upgrade to a paid subscription (clarify that having a payment method on file does not trigger a charge), or split into two sentences that separately state the automatic reversion and the charging behavior depending on whether the user upgraded.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/importers/flatfile.md`:
- Line 122: Update the table cell that currently reads "Created By | **Must be
email addresses** - allows Plane to track who originally created the item" to
the tightened wording "Created By | **Must be email addresses** - allows Plane
to track who created the item"; locate the string in docs/importers/flatfile.md
(the "Created By" table row) and replace the redundant phrase "originally
created the item" with "created the item".
- Around line 46-50: Remove trailing whitespace on the line containing the
"Sample CSV template" tip link and delete the whitespace-only blank line
immediately after it so the markdown block contains no trailing or blank-only
lines; locate the tip block that starts with "::: tip Sample CSV template" and
the link to "sample CSV file" and clean up the two offending whitespace
characters to satisfy the formatter.
---
Nitpick comments:
In `@docs/workspaces-and-users/billing-and-plans.md`:
- Line 153: The sentence "After 14 days, your workspace automatically reverts to
the Free plan if you haven't upgraded to a paid subscription, and you won’t be
charged if a payment method was never added." is ambiguous about the case where
a payment method was added during the trial but no upgrade occurred; rewrite the
line (the sentence starting with "After 14 days...") to explicitly cover both
outcomes—e.g., state that the workspace reverts to Free after 14 days and that
users will not be charged unless they actively upgrade to a paid subscription
(clarify that having a payment method on file does not trigger a charge), or
split into two sentences that separately state the automatic reversion and the
charging behavior depending on whether the user upgraded.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 26e01c86-0bde-431a-bd6b-1b6cbb0e3e5e
📒 Files selected for processing (3)
docs/importers/csv.mddocs/importers/flatfile.mddocs/workspaces-and-users/billing-and-plans.md
| | State | Must map to valid states in your Plane project | | ||
| | Assignees | **Must be email addresses**, not usernames - this is how Plane connects work items to users | | ||
| | Priority | Must map to valid priority values in your Plane project | | ||
| | Created By | **Must be email addresses** - allows Plane to track who originally created the item | |
There was a problem hiding this comment.
Tighten redundant phrasing in field note.
Line 122 reads a bit verbose (“originally created the item”). Consider “track who created the item” for clarity and brevity.
🧰 Tools
🪛 LanguageTool
[style] ~122-~122: This phrase is redundant. Consider writing “created”.
Context: ...addresses** - allows Plane to track who originally created the item | | St...
(ORIGINALLY_CREATED)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/importers/flatfile.md` at line 122, Update the table cell that currently
reads "Created By | **Must be email addresses** - allows Plane to track who
originally created the item" to the tightened wording "Created By | **Must be
email addresses** - allows Plane to track who created the item"; locate the
string in docs/importers/flatfile.md (the "Created By" table row) and replace
the redundant phrase "originally created the item" with "created the item".
sriramveeraghanta
approved these changes
Mar 27, 2026
There was a problem hiding this comment.
♻️ Duplicate comments (1)
docs/importers/flatfile.md (1)
120-120:⚠️ Potential issue | 🟡 MinorTighten redundant phrasing.
The phrase "originally created the item" is redundant. Consider "track who created the item" for clarity and brevity.
✂️ Proposed simplification
-| Created By | **Must be email addresses** - allows Plane to track who originally created the item | +| Created By | **Must be email addresses** - allows Plane to track who created the item |🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/importers/flatfile.md` at line 120, Update the table cell text to remove redundant phrasing: replace "allows Plane to track who originally created the item" with "allows Plane to track who created the item" so the full row reads "Created By | **Must be email addresses** - allows Plane to track who created the item"; keep the emphasis and requirement about email addresses unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Duplicate comments:
In `@docs/importers/flatfile.md`:
- Line 120: Update the table cell text to remove redundant phrasing: replace
"allows Plane to track who originally created the item" with "allows Plane to
track who created the item" so the full row reads "Created By | **Must be email
addresses** - allows Plane to track who created the item"; keep the emphasis and
requirement about email addresses unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: e34522e6-d9e4-41a2-885c-01ad381d8f12
📒 Files selected for processing (1)
docs/importers/flatfile.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Type of Change
Screenshots and Media (if applicable)
Test Scenarios
References
Summary by CodeRabbit