Skip to content

fix: merging dev to main#133

Merged
Avijit-Microsoft merged 4 commits intomainfrom
dev
May 21, 2025
Merged

fix: merging dev to main#133
Avijit-Microsoft merged 4 commits intomainfrom
dev

Conversation

@Roopan-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

  • This pull request introduces a new environment variable, REACT_APP_AUTH_ENABLED, to enable or disable authentication features across the application. The most important changes include adding conditional logic to check the value of this variable and modifying components to respect the authentication toggle.

Authentication Toggle Implementation:

  • Environment Variable Addition:

    • Added REACT_APP_AUTH_ENABLED to the .env file to allow toggling authentication. Defaults to true if not set. (src/ContentProcessorWeb/.env, src/ContentProcessorWeb/.envR14-R15)
  • Header Component Updates:

    • Introduced authEnabled in HeaderPage to conditionally render the tools section based on the authentication toggle. (src/ContentProcessorWeb/src/Components/Header/Header.tsx, [1] [2] [3]
  • Authentication Wrapper Updates:

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

@Avijit-Microsoft Avijit-Microsoft merged commit d62a182 into main May 21, 2025
7 of 8 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

4 participants