viewMapped retain required fields option#769
Conversation
71c46ea to
c9c963a
Compare
elisadinsmore
left a comment
There was a problem hiding this comment.
Tested and approved!
|
Caution Review failedThe pull request is closed. WalkthroughThis update introduces a new option for the Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Plugin
participant FieldProcessor
Client->>+Plugin: viewMappedPlugin(options)
Plugin->>Plugin: Begin iterating over workbook fields
alt Field is mapped
Plugin->>FieldProcessor: Retain field
else Field is not mapped
alt options.keepRequiredFields enabled and field required
Plugin->>FieldProcessor: Retain required field
else
Plugin->>FieldProcessor: Exclude field
end
end
Plugin-->>-Client: Return filtered fields
Possibly related PRs
Suggested Reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
📝 Documentation updates detected! You can review documentation updates here |
Please explain how to summarize this PR for the Changelog:
Tell code reviewer how and what to test: