Skip to content

Standardize export sheet name - #2

Open
reilly-webster wants to merge 7 commits into
2026.xfrom
feature/standarize-export-sheet-names
Open

Standardize export sheet name#2
reilly-webster wants to merge 7 commits into
2026.xfrom
feature/standarize-export-sheet-names

Conversation

@reilly-webster

Copy link
Copy Markdown

This pull request adds support for specifying a custom sheet name when exporting data to XLSX files and updates related UI and backend logic. It also includes minor improvements to translation strings for consistency.

XLSX Export Enhancements:

  • Added a new sheetName field to the XLSX export form, allowing users to specify the name of the sheet in the exported file (create-xlsx-form.tsx, XLSXFormValues) [1] [2].
  • Updated the XLSX modal logic to initialize the sheetName field with a default value (Sheet1) and to pass this value through the export job and backend request (xlsx-modal.tsx) [1] [2] [3] [4].
  • Added a translation key for the new sheet name field label (studio.en.yaml).

Translation and Formatting Improvements:

  • Standardized the use of single quotes for YAML string values containing double quotes for better consistency and to avoid parsing issues (studio.en.yaml) [1] [2].## Changes in this pull request

},
header,
sheetName
} as any,

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.

Do we just need to update the config type to include sheetName as an available config field so we don't need any?

},
header,
sheetName
} as any,

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.

same here

header
},
header,
sheetName

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.

We probably also need a pull request to the studio backend bundle to also use this sheetname param.

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.

3 participants