Skip to content

Conversation

@gmasclet
Copy link
Contributor

@gmasclet gmasclet force-pushed the LKE-14349-import-template branch 2 times, most recently from bb64dc1 to 8683ffe Compare November 28, 2025 15:20
@gmasclet gmasclet force-pushed the LKE-14349-import-template branch from 8683ffe to 22351bd Compare December 2, 2025 10:13
Comment on lines 49 to 55
* The field in the imported file.
*/
sourceField: string;
/**
* The destination node category.
*/
targetCategory: string;
Copy link
Contributor

@WilliamGorge WilliamGorge Dec 4, 2025

Choose a reason for hiding this comment

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

let's use better names to avoid having "source" and "target" twice

Suggested change
* The field in the imported file.
*/
sourceField: string;
/**
* The destination node category.
*/
targetCategory: string;
* The field in the imported file.
*/
importedFileField: string;
/**
* The destination node category.
*/
destinationNodeCategory: string;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

indeed, it's done 👍

@gmasclet gmasclet requested a review from WilliamGorge December 4, 2025 15:09
@gmasclet
Copy link
Contributor Author

gmasclet commented Dec 4, 2025

@WilliamGorge PR ready for re-review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants