Skip to content

Move the BulkImportService to BulkImports::CreateService

João Alexandre Cunha requested to merge jcunha-extrac-controller-logic-to-service into master

What does this MR do and why?

This is a refactor to follow our guidelines on "Use namespaces to define bounded contexts"

# Commit message
Refactor the BulkImportService

    It's better to have feature specific logic scoped under their own
    namespaces instead of the root namespace. So here we're moving:
    BulkImportService to BulkImports::CreateService.

    I also intend to create other services for the BulkImports. For instance,
    the BulkImports::StatusService which will allows us to create logic for the
    status and not have to shove them into the BulkImportsController.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by João Alexandre Cunha

Merge request reports