Altinn / app-template-dotnet

Altinn Studio application template
1 stars 8 forks source link

Move EFormidling logic out of AppBase #115

Open RonnyB71 opened 2 years ago

RonnyB71 commented 2 years ago

Goal: Register, configure and inject EFormidling only when the app actually uses it. EFormidling should be a separate nuget package and is a good candidate for testing that an "add-on" like this can be configured only through importing the package and register the necessary services. This means we need to have interfaces defined that allows EFormidling to be injected and executed at the right places ref. Move from AppBase... task.

Update 31.08.2022

Eformidling functionality is to some extent moved out by by the recent changes in V7 development, but it's still integrated in the Core project with dependencies to our DefaultTaskEvent implementation. Issue is not strictly required to get V7 out the door and should therefor be solve as a separate issue after the V7 release

Tasks

tjololo commented 2 years ago

In branch https://github.com/Altinn/app-lib-dotnet/tree/chore/90-114-event-pattern I have a suggestion for an event pattern. This change lead to the removal of EFormidling from AppBase, it's not a separate nuget, but all the code is located in a separate namespace