Skip to content

Auth0 refactor: Unify Auth0 Codebase#4

Open
MigangalWork wants to merge 1 commit intoneuraptic:mainfrom
MigangalWork:refactor/auth0
Open

Auth0 refactor: Unify Auth0 Codebase#4
MigangalWork wants to merge 1 commit intoneuraptic:mainfrom
MigangalWork:refactor/auth0

Conversation

@MigangalWork
Copy link
Copy Markdown
Contributor

@MigangalWork MigangalWork commented Dec 18, 2024

Background

Our current Auth0 implementation is scattered across the codebase due to time constraints during initial development. This dispersion makes it challenging to maintain and test.
This pull request aims to address these issues by unifying all Auth0 connection code into a single module, improving maintainability and enhancing code comprehension.

Changes

Major Features.

  • Introduced a new external folder to organize and encapsulate all external connection-related code.
  • Added an nexusml.api.external.auth0 module containing the Auth0Manager class to centralize and streamline all Auth0-related functionality.

Refactoring.

  • Refactored existing code to align with the new Auth0 structure, ensuring consistency throughout the project.

Tests.

  • Resolved an issue in the tests where old roles were not being deleted before adding new ones.

@MigangalWork MigangalWork marked this pull request as draft December 18, 2024 10:59
- Introduced a new `external` folder to organize and encapsulate all external connection-related code.
- Added an `Auth0` module containing the `Auth0Manager` class to centralize and streamline all Auth0-related functionality.
- Refactored existing code to align with the new Auth0 structure, ensuring consistency throughout the project.
- Resolved an issue in the tests where old roles were not being deleted before adding new ones.
@MigangalWork MigangalWork marked this pull request as ready for review December 18, 2024 11:04
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