Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

520 fix an issue with the export #535

Merged
merged 5 commits into from
Jan 22, 2025
Merged

Conversation

kuh0005
Copy link
Collaborator

@kuh0005 kuh0005 commented Jan 22, 2025

This pull request includes several changes to improve the data export and import functionality in the AXOpen.Data.Blazor project. The changes mainly focus on enhancing the user interface and handling the status of export and import operations more effectively.

Enhancements to Export and Import Functionality:

  • src/data/app/ix-blazor/librarytemplate.blazor/librarytemplate.blazor.csproj: Added a project reference to AXOpen.Data.Exporters.Excel.csproj to support Excel export functionality.

  • src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeView.razor:

    • Simplified the Export and Import button click handlers to only load custom export data.
    • Updated the export modal to handle different export statuses (Busy, Done, Failed) and added detailed export options. [1] [2] [3]
    • Improved the import modal to handle import statuses and display appropriate messages. [1] [2]

Codebase Improvements:

  • src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeView.razor.cs:

    • Introduced properties to manage file paths and statuses for export and import operations.
    • Added methods to clear client files and reset statuses.
    • Updated LoadFile method to handle file loading status.
  • src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeViewModel.cs:

    • Replaced IsFileExported with exportStatus and importStatus to track operation statuses more accurately.
    • Updated ExportDataAsync and ImportDataAsync methods to use the new status properties. [1] [2]
    • Changed FirstNumber and SecondNumber in ExportSettings to uint for better type safety.

@kuh0005 kuh0005 linked an issue Jan 22, 2025 that may be closed by this pull request
@PTKu PTKu merged commit d1d033c into dev Jan 22, 2025
1 check passed
@PTKu PTKu deleted the 520-fix-an-issue-with-the-export branch January 22, 2025 14:30
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.

Fix an issue with the export
2 participants