Skip to content

Added csv edit modal#3351

Open
Corepex wants to merge 4 commits into2025.4from
3165-select-options---csv-option-is-missing
Open

Added csv edit modal#3351
Corepex wants to merge 4 commits into2025.4from
3165-select-options---csv-option-is-missing

Conversation

@Corepex
Copy link
Copy Markdown
Contributor

@Corepex Corepex commented Apr 17, 2026

Changes in this pull request

Resolves #3165

Additional info

This pull request introduces a CSV import feature to the Operational Grid component, allowing users to import and edit grid data via a CSV modal dialog. The feature can be enabled or disabled via a prop, and includes UI integration, context wiring, and localization.

CSV Import Feature Integration:

  • Added a new CsvImportModal React component for editing grid data as CSV, with conversion logic between rows and CSV text.
  • Updated the useOperations hook to manage the CSV import modal's open state, handle confirmation/cancellation, and expose openCsvImport and csvImportModal for use in UI components. [1] [2] [3] [4]

Component and Storybook Updates:

  • Integrated the CSV import button and modal into the Operational Grid's UI and Storybook stories, with conditional rendering based on feature availability. [1] [2] [3] [4] [5]

Configuration and Context Wiring:

  • Added a disableCsvImport prop to OperationalGridProps and passed it through the context provider, allowing feature toggling at the component level. [1] [2] [3]

Localization:

  • Added translation keys for the CSV import modal's UI text to the English translation file.

@Corepex Corepex requested a review from xIrusux April 17, 2026 07:04
@Corepex Corepex self-assigned this Apr 17, 2026
@Corepex Corepex linked an issue Apr 17, 2026 that may be closed by this pull request
return (
<OperationalGridProvider
columns={ columns }
disableCsvImport={ disableCsvImport }
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Randomly came across this one. It feels a bit strange to see these changes directly in the operational grid. Can't we just an button that then consumes the necessary information from the context instead?

At least I wouldn't change the behavior of the operational grid here, since it does not feel like something that is directly connected to the API of the grid.

@sonarqubecloud
Copy link
Copy Markdown

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.

[Select options] - csv option is missing

2 participants