Skip to content

Conversation

@Kanchan-Microsoft
Copy link
Contributor

Purpose

This pull request includes a variety of cleanup, bugfixes, and code simplification changes across the backend Python code, frontend React application, and deployment/documentation files. The main focus is on removing unused code, fixing minor bugs, clarifying documentation, and improving maintainability.

Backend Python code improvements:

  • Changed parse_polygon field validator methods in Word, Line, and Paragraph classes to use self instead of cls for correct method signatures.
  • Updated save_to_persistent_storage in pipeline message and status classes to accept account_url and container_name parameters, clarifying intended usage and removing unimplemented pass statements.
  • Fixed parameter name in download_file method from file_path to download_path to match the expected argument in StorageBlobHelper.
  • Removed an unused internal helper function (_find_process_result) from queue_handler_base.py for clarity.
  • Removed unnecessary variable assignment in remote_module_loader.py.
  • Updated placeholder dependency functions in FastAPI to raise exceptions instead of using pass, making error handling explicit.

Frontend React code cleanup and simplification:

  • Removed unused imports and variables across multiple components, including Header.tsx, UploadFilesModal.tsx, ProcessQueueGrid.tsx, ProcessSteps.tsx, and others, to reduce bundle size and improve readability.
  • Removed or updated unnecessary state and props, such as removing unused fileType and useDispatch in process queue and process steps components.
  • Fixed minor bugs, such as correcting function formatting and removing references to deleted variables.

Documentation and deployment script updates:

  • Removed obsolete "Known Issues" and workaround section from DeploymentGuide.md as Azure Portal issues have been resolved.
  • Fixed a small typo in the deployment guide tip.
  • Removed the SecurityControl tag from the Azure deployment workflow as it is no longer needed.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

Roopan-Microsoft and others added 8 commits December 15, 2025 11:43
Removed known issues section regarding environment variables and container configuration in Azure Container Apps.
…ues-azure

docs: Remove known issues about Azure Container Apps
…ntrol-tag

fix: Removed SecurityControl tag
@Prajwal-Microsoft Prajwal-Microsoft merged commit 143de68 into microsoft:dev Dec 24, 2025
5 checks passed
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.

5 participants