Skip to content

Conversation

@kavehshahedi
Copy link
Contributor

What it does

Introduce a new interface and model class to allow data providers to fetch and return custom data based on provider configuration.

  • ITmfDataProviderConfigurationDataFetcher interface that allows data providers to implement custom data retrieval capabilities without requiring specific knowledge of the data format in the core API.
  • TmfDataProviderConfigurationDataModel class to encapsulate different types of content (files, byte arrays, strings, etc.) along with associated metadata such as content type and filename.

This extension enables data providers to serve various types of content (reports, images, data exports, etc.) in a standardized way while keeping the core API independent of specific implementations.

How to test

Run the project tests via mvn clean test.

Follow-ups

Any data provider configuration that needs to have such a functionality can implement the ITmfDataProviderConfigurationDataFetcher interface and provide the required logic in order to retrieve its data.

Review checklist

  • As an author, I have thoroughly tested my changes and carefully followed the instructions in this template

@kavehshahedi kavehshahedi force-pushed the configuration-data-fetch branch 4 times, most recently from 1345446 to 27a6833 Compare April 10, 2025 18:44
MatthewKhouzam
MatthewKhouzam previously approved these changes Apr 10, 2025
Introduce a new interface ITmfDataProviderConfigurationDataFetcher and
model class TmfDataProviderConfigurationDataModel to allow data providers
to fetch and return custom data based on provider configuration.

[Added] Infrastructure of fetching a data provider configuration's data

Signed-off-by: Kaveh Shahedi <kaveh.shahedi@ericsson.com>
@kavehshahedi kavehshahedi force-pushed the configuration-data-fetch branch from 9fe4441 to ee9fec3 Compare April 17, 2025 14:31
@kavehshahedi kavehshahedi requested a review from bhufmann April 17, 2025 15:03
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.

3 participants