Skip to content

[Node] fix multiple template+CSV+manual edit import issues - #3645

Open
GuillaumeDSM wants to merge 1 commit into
devfrom
node_import
Open

[Node] fix multiple template+CSV+manual edit import issues#3645
GuillaumeDSM wants to merge 1 commit into
devfrom
node_import

Conversation

@GuillaumeDSM

Copy link
Copy Markdown
Member
Scenario Before After
Blank blockchain_from cell + XXX template Empty Transfer Network Defaults to template value
Wrong auto-detected template value blockchain_from unmapped Import JSON / Apply to all rows / pick template-value
Review → Back Edits lost, re-detected Edits restored from parent state
Change template after editing non template-valued fields CSV value restored Manual edit kept; only template-enforced fields update

@GuillaumeDSM GuillaumeDSM self-assigned this Aug 14, 2026
@GuillaumeDSM
GuillaumeDSM requested a review from Herklos as a code owner August 14, 2026 09:07
@GuillaumeDSM
GuillaumeDSM enabled auto-merge (rebase) August 14, 2026 09:19
unmappedColumns: number[],
): number[] {
const nameColumnIndex = headers.findIndex(
(header) => header.trim().toLowerCase() === "name",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't we prefer a param for header col name and so have filterFromUnmappedColumns instead of filterNameFromUnmappedColumns

rows: string[][],
detectionResults: ReturnType<typeof detectColumnsAndTemplates>,
): ActionRow[] {
const nameIdx = headers.findIndex((h) => h.trim().toLowerCase() === "name")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same here

@GuillaumeDSM

Copy link
Copy Markdown
Member Author

thanks, it's up!

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