Merge branch 'fix-settings-errors' into 'release/2.4.3-rc1'

make sure setSettings returns expected info

See merge request !24

src/app/services/settings.service.ts

@@ -23,7 +23,12 @@ export class SettingsService {
   public setSettings(settings: Setting): Observable<Setting> {
     return this.http.post<Setting>(this.configService.config.url + '/services/settings', settings, {observe: "response"}).pipe(
       map(response => {
-        return response.body;
+        if (response.status === 204) {
+          // this is a post which returns nothing, can't use response.body
+          return settings;
+        }
+        // if for some completely weird reason, we do have response content, send it on, it's probably an error
+        return response.body
       }));
   }