Skip to content

Fix/producer impl - #247

Open
MinetteMeyo wants to merge 4 commits into
developfrom
fix/producerImpl
Open

Fix/producer impl#247
MinetteMeyo wants to merge 4 commits into
developfrom
fix/producerImpl

Conversation

@MinetteMeyo

Copy link
Copy Markdown
Collaborator
  • Refactored AbstractProducer and all related classes, i.e., base producers, to clean unused code and remove redundant methods.
  • Renamed all the concrete producers with their base producers' prefix. E.g., BTUProducer became CSVBTUProducer, and the same for tests and associated queries
  • Fixed the issue with the absolute path in the repo. Two routes were defined in the application.properties file, and the ' AbstractProducer ' method was updated accordingly.
  • Created a complete pipeline of to access TXT files from the hub in the middleware. All the queries rely on POST {baseurl}/apply/sync or async. All TXT files will rely on the TXTProducer; only the path (actually the filename) will differ

…e producer and one concrete producer, alongside a Test and a query
…ionProducer, and creation of the producerDataController
…base producers' prefix, updating the TXTProducer pipeline entirely, and cleaning a few tests
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