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.

Sign up for GitHub

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

Jump to bottom

Remove Unnecessary Class Instances #474

Open
ashleyshaw opened this issue Jan 9, 2025 · 0 comments
Open

Remove Unnecessary Class Instances #474

ashleyshaw opened this issue Jan 9, 2025 · 0 comments
Assignees
@krugazul
Labels
[Type] Enhancement A suggestion for improvement
Milestone
2.1.0

Comments

@ashleyshaw
Copy link
Member

Description
Reduce or remove extra class instances in the plugin. Some classes may be instantiated multiple times unnecessarily, leading to complexity and confusion.

Why This Matters

  • Simplifies the code structure, making it easier to maintain and test.
  • Aligns with best practices for singleton usage and code clarity.

Tasks

  1. Audit new ClassName() calls throughout the code.
  2. Consolidate or remove superfluous instances.
  3. Ensure no existing features or flows break after changes.
@ashleyshaw ashleyshaw added the [Type] Enhancement A suggestion for improvement label Jan 9, 2025
@ashleyshaw ashleyshaw added this to the 2.1.0 milestone Jan 9, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Tour Operator Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@krugazul krugazul

Labels
[Type] Enhancement A suggestion for improvement
Projects
Tour Operator
Status: Needs Triage
Milestone
2.1.0
Development

No branches or pull requests
2 participants
@ashleyshaw @krugazul