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

Minor Restructuring #18

Open
yatharthranjan opened this issue Jul 5, 2020 · 0 comments
Open

Minor Restructuring #18

yatharthranjan opened this issue Jul 5, 2020 · 0 comments

Comments

@yatharthranjan
Copy link
Member

A minor restructuring is required on how the configurations are handled in the data uploader code. These need to be simplified.

  • The Data consumer can be a singleton and a factory can provide the instance of each type. Removes the need for using reflection and static configuration.
  • The sensor configs can be handled in one class called SensorConfigs and don't need to be kept untidy using maps.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yatharthranjan