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

IConfigurationFactory cannot be considered as the loosely coupled part #402

Open
mpostol opened this issue Nov 27, 2019 · 1 comment
Open

IConfigurationFactory cannot be considered as the loosely coupled part #402

mpostol opened this issue Nov 27, 2019 · 1 comment
Assignees
@mpostol
Labels
Documentation Documentation modification request
Milestone
Networking backlog

Comments

@mpostol
Copy link
Owner

mpostol commented Nov 27, 2019

The interface should be a contract to be used by Dependency Injection. It is hard to implement because of the IConfigurationFactory:

  • implementation is in the same assembly
  • refers directly to the type ConfigurationData

Add documentation on how to make custom implementation loosely coupled.

It is related to Drutol/CrossHMI#7
@mpostol mpostol added the Documentation Documentation modification request label Nov 27, 2019
@mpostol mpostol added this to the Review the documentation milestone Nov 27, 2019
@mpostol mpostol added this to Backlog in DataDiscovery via automation Nov 27, 2019
@mpostol mpostol self-assigned this Nov 27, 2019
@mpostol
Copy link
Owner Author

mpostol commented Dec 3, 2019

First approach to work out a domain model:
2019-12-03_08-52-01

@mpostol mpostol removed this from Backlog in DataDiscovery Dec 9, 2019
@mpostol mpostol added this to To do in Networking.ReactiveHMI via automation Dec 9, 2019
@mpostol mpostol removed this from To do in Networking.ReactiveHMI Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mpostol mpostol

Labels
Documentation Documentation modification request
Projects
None yet
Milestone
Networking backlog
Development

No branches or pull requests
1 participant
@mpostol