search

episerver / upgrade-assistant-extensions

Apache License 2.0
10 stars 6 forks source link

Missing configs for CMS #13

Closed LucGosso closed 2 years ago

LucGosso commented 2 years ago

Suggestion, when someone has time:

would it be possible to add:

  1. .ConfigureCmsDefaults() in program.cs 2. add ConnectionStrings to appsettings.json (port from web.config)
  2. add ConfigureApplicationCookie to Startup.cs

For a smoother porting

lunchin commented 2 years ago

Connectionstrings should be ported by upgrade-assistant. The others can update

JohanPetersson commented 2 years ago

Why is ConfigureApplicationCookie needed? Seems specific to your application and should not be needed.

lunchin commented 2 years ago

@LucGosso do you still want configure application cookie. Seems the others were fixed with bug fix contribution we made a couple months ago.

LucGosso commented 2 years ago

nope, just suggestion, see ya