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

[IMP] Remove inter module dependency #274

Merged
merged 23 commits into from
Aug 5, 2024
Merged

Conversation

horilla-opensource
Copy link
Owner

This commit introduces significant changes to the architecture of the Horilla HRMS system by decoupling interdependent modules. The following modifications were made:

  1. Module Independence: Each module has been refactored to eliminate reliance on other modules, promoting a more modular and maintainable codebase.
  2. Refactored Imports and Dependencies: Adjusted import statements and dependency injections to support independent module operation.
  3. Compatibility and Functionality: Ensured that all modules are compatible with existing systems and maintain their intended functionality both independently and when integrated with other modules.

These changes enhance the modularity, maintainability, and scalability of the Horilla HRMS, allowing developers to work on individual modules without affecting the entire system. Future development and deployment will be more efficient and less prone to issues arising from tightly coupled code.

NOTE
For existing Horilla users, if you face any issues during the migrations, please run the following command and try again the migrations.

  • python3 manage.py makemigrations
  • python3 manage.py migrate base
  • python3 manage.py migrate

horilla-opensource and others added 23 commits August 3, 2024 16:08
Remove dummy db init password
@horilla-opensource horilla-opensource merged commit 2fee7c1 into master Aug 5, 2024
1 check failed
Repository owner deleted a comment from gitguardian bot Aug 5, 2024
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