acm-udayton / acm-udayton/ACM-Meeting-Records

[TASK] Migrate business logic to services

Offen
#160 0 Kommentare 0 Reaktionen 2 zugewiesene Personen Beansprucht von @ethanlally Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Python
Sterne
7
Forks
2
Ø Merge
4 T. 23 Std.
Gemergte PRs (30 T.)
2

Beschreibung

## Feature Description

Business logic should be moved from within the routes and blueprints to separate services. These services should be contained in a `app/services` directory, with files mirroring the blueprints where they would have previously been located.

## Why It's Worth Implementing

Not only does this make our app more consistent with best practices and increase maintainability and testing ease, it also is key to our process of decoupling the frontend and backend. These services will allow us to keep consistent business logic during the intermediate phase where we keep the existing Jinja template serving and also build out the RESTful API.

---

## Acceptance Criteria & Solution Requirements
*To close this issue, the following must be met (can be finalized in the comments):*
- [x] 1. Application runs as expected with the new feature integrated seamlessly.
- [x] 2. Code quality metrics are maintained, and linting has been considered in development.
- [x] 3. Testing metrics are maintained, and tests have been updated/added wherever relevant during development.
- [ ] 4. Documentation has been updated to reflect the changes made during development.
- [ ] 5. All business logic has been removed from the `app/blueprints` directory, and is instead kept in the `app/services` directory.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.