Skip to content

Issues/75#333

Merged
MayGo merged 4 commits intomasterfrom
issues/75
Mar 20, 2025
Merged

Issues/75#333
MayGo merged 4 commits intomasterfrom
issues/75

Conversation

@MayGo
Copy link
Owner

@MayGo MayGo commented Mar 20, 2025

  • Updated the export button to open a menu for selecting export formats (CSV and JSON).
  • Modified the exportItems function to accept a format parameter, defaulting to CSV.
  • Adjusted the API and service layers to handle JSON export, including proper file content generation and extension handling.
  • Updated tests to reflect changes in the export functionality and ensure both formats are correctly processed.

MayGo added 4 commits March 20, 2025 12:48
- Adjusted the toDateTimeStr function to parse timestamps as integers for accurate formatting.
- Modified the default CSV file path generation to use moment for consistent date formatting.
- Updated the export button to open a menu for selecting export formats (CSV and JSON).
- Modified the exportItems function to accept a format parameter, defaulting to CSV.
- Adjusted the API and service layers to handle JSON export, including proper file content generation and extension handling.
- Updated tests to reflect changes in the export functionality and ensure both formats are correctly processed.
- Marked the test for exporting items to CSV and JSON as skipped to prevent it from running until further implementation or fixes are made.
- Added a mock for the scrollTo method to prevent errors with React 19 and Chakra UI.
- Enabled the previously skipped test for exporting items to CSV and JSON, allowing it to run and validate the export functionality.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'maygo_tockler_electron'

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

@MayGo MayGo merged commit 31d393c into master Mar 20, 2025
7 of 8 checks passed
@MayGo MayGo deleted the issues/75 branch March 20, 2025 11:42
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.

1 participant