bot/src/bot_api/controller/auth_controller.py

@@ -14,10 +14,7 @@ from bot_api.model.reset_password_dto import ResetPasswordDTO
 from bot_api.model.token_dto import TokenDTO
 from bot_api.model.update_auth_user_dto import UpdateAuthUserDTO
 from bot_api.route.route import Route
-from bot_core.configuration.feature_flags_enum import FeatureFlagsEnum
-from bot_core.configuration.feature_flags_settings import FeatureFlagsSettings
 from bot_data.model.auth_role_enum import AuthRoleEnum
-from bot_data.model.technician_config import TechnicianConfig
 
 
 class AuthController:
@@ -33,7 +30,6 @@ class AuthController:
         mail_settings: EMailClientSettings,
         mailer: EMailClientABC,
         auth_service: AuthServiceABC,
-        technician_config: TechnicianConfig,
     ):
         self._config = config
         self._env = env
@@ -43,7 +39,6 @@ class AuthController:
         self._mail_settings = mail_settings
         self._mailer = mailer
         self._auth_service = auth_service
-        self._technician_config = technician_config
 
     @Route.get(f"{BasePath}/users")
     @Route.authorize(role=AuthRoleEnum.admin)
@@ -75,32 +70,17 @@ class AuthController:
 
     @Route.post(f"{BasePath}/register")
     async def register(self):
-        if not FeatureFlagsSettings.get_flag_from_dict(
-            self._technician_config.feature_flags, FeatureFlagsEnum.basic_registration
-        ):
-            return
-
         dto: AuthUserDTO = JSONProcessor.process(AuthUserDTO, request.get_json(force=True, silent=True))
         self._auth_service.add_auth_user(dto)
         return "", 200
 
     @Route.post(f"{BasePath}/register-by-id/<id>")
     async def register_id(self, id: str):
-        if not FeatureFlagsSettings.get_flag_from_dict(
-            self._technician_config.feature_flags, FeatureFlagsEnum.basic_registration
-        ):
-            return
-
         result = await self._auth_service.confirm_email_async(id)
         return jsonify(result)
 
     @Route.post(f"{BasePath}/login")
     async def login(self) -> Response:
-        if not FeatureFlagsSettings.get_flag_from_dict(
-            self._technician_config.feature_flags, FeatureFlagsEnum.basic_login
-        ):
-            return jsonify({})
-
         dto: AuthUserDTO = JSONProcessor.process(AuthUserDTO, request.get_json(force=True, silent=True))
         result = await self._auth_service.login_async(dto)
         return jsonify(result.to_dict())
@@ -120,11 +100,6 @@ class AuthController:
 
     @Route.post(f"{BasePath}/forgot-password/<email>")
     async def forgot_password(self, email: str):
-        if not FeatureFlagsSettings.get_flag_from_dict(
-            self._technician_config.feature_flags, FeatureFlagsEnum.basic_login
-        ):
-            return "", 409
-
         await self._auth_service.forgot_password_async(email)
         return "", 200
 
@@ -135,11 +110,6 @@ class AuthController:
 
     @Route.post(f"{BasePath}/reset-password")
     async def reset_password(self):
-        if not FeatureFlagsSettings.get_flag_from_dict(
-            self._technician_config.feature_flags, FeatureFlagsEnum.basic_login
-        ):
-            return "", 409
-
         dto: ResetPasswordDTO = JSONProcessor.process(ResetPasswordDTO, request.get_json(force=True, silent=True))
         await self._auth_service.reset_password_async(dto)
         return "", 200

bot/src/bot_api/controller/gui_controller.py

@@ -12,9 +12,6 @@ from bot_api.logging.api_logger import ApiLogger
 from bot_api.model.settings_dto import SettingsDTO
 from bot_api.model.version_dto import VersionDTO
 from bot_api.route.route import Route
-from bot_core.configuration.feature_flags_enum import FeatureFlagsEnum
-from bot_core.configuration.feature_flags_settings import FeatureFlagsSettings
-from bot_data.model.technician_config import TechnicianConfig
 
 
 class GuiController:
@@ -85,11 +82,3 @@ class GuiController:
         )
         self._mailer.send_mail(mail)
         return "", 200
-
-    @Route.get(f"{BasePath}/has-feature-flag/<flag>")
-    async def has_feature_flag(self, flag: str):
-        settings: TechnicianConfig = self._config.get_configuration(TechnicianConfig)
-        return {
-            "key": flag,
-            "value": FeatureFlagsSettings.get_flag_from_dict(settings.feature_flags, FeatureFlagsEnum(flag)),
-        }

bot/src/bot_core/configuration/feature_flags_enum.py

@@ -28,5 +28,3 @@ class FeatureFlagsEnum(Enum):
     technician_full_access = "TechnicianFullAccess"
     steam_special_offers = "SteamSpecialOffers"
     scheduled_events = "ScheduledEvents"
-    basic_registration = "BasicRegistration"
-    basic_login = "BasicLogin"

bot/src/bot_core/configuration/feature_flags_settings.py

@@ -30,8 +30,6 @@ class FeatureFlagsSettings(ConfigurationModelABC):
         FeatureFlagsEnum.technician_full_access.value: False,  # 03.10.2023 #393
         FeatureFlagsEnum.steam_special_offers.value: False,  # 11.10.2023 #188
         FeatureFlagsEnum.scheduled_events.value: False,  # 14.11.2023 #410
-        FeatureFlagsEnum.basic_registration.value: False,  # 19.11.2023 #440
-        FeatureFlagsEnum.basic_login.value: False,  # 19.11.2023 #440
     }
 
     def __init__(self, **kwargs: dict):

bot/src/bot_graphql/graphql/query.gql

@@ -50,6 +50,4 @@ type Query {
     technicianConfig: TechnicianConfig
     possibleFeatureFlags: [String]
     discord: Discord
-
-    hasFeatureFlag(flag: String): FeatureFlag
 }

bot/src/bot_graphql/query.py

@@ -1,9 +1,7 @@
-from cpl_core.configuration import ConfigurationABC
 from cpl_discord.service import DiscordBotServiceABC
 from cpl_query.extension import List
 
 from bot_core.configuration.feature_flags_enum import FeatureFlagsEnum
-from bot_core.configuration.feature_flags_settings import FeatureFlagsSettings
 from bot_data.abc.achievement_repository_abc import AchievementRepositoryABC
 from bot_data.abc.auto_role_repository_abc import AutoRoleRepositoryABC
 from bot_data.abc.client_repository_abc import ClientRepositoryABC
@@ -24,7 +22,6 @@ from bot_data.abc.user_joined_voice_channel_repository_abc import (
 from bot_data.abc.user_repository_abc import UserRepositoryABC
 from bot_data.abc.user_warnings_repository_abc import UserWarningsRepositoryABC
 from bot_data.model.short_role_name_position_enum import ShortRoleNamePositionEnum
-from bot_data.model.technician_config import TechnicianConfig
 from bot_graphql.abc.query_abc import QueryABC
 from bot_graphql.filter.achievement_filter import AchievementFilter
 from bot_graphql.filter.auto_role_filter import AutoRoleFilter
@@ -48,7 +45,6 @@ from modules.achievements.achievement_service import AchievementService
 class Query(QueryABC):
     def __init__(
         self,
-        config: ConfigurationABC,
         bot: DiscordBotServiceABC,
         auto_roles: AutoRoleRepositoryABC,
         clients: ClientRepositoryABC,
@@ -69,8 +65,6 @@ class Query(QueryABC):
     ):
         QueryABC.__init__(self, "Query")
 
-        self._config = config
-
         self.add_collection("autoRole", lambda *_: auto_roles.get_auto_roles(), AutoRoleFilter)
         self.add_collection(
             "autoRoleRule",
@@ -126,17 +120,3 @@ class Query(QueryABC):
 
         self.set_field("possibleFeatureFlags", lambda *_: [e.value for e in FeatureFlagsEnum])
         self.set_field("discord", lambda *_: Discord(bot.guilds, List(any).extend(bot.users)))
-
-        self.set_field(
-            "hasFeatureFlag",
-            lambda *_, **kwargs: self._resolve_has_feature_flag(*_, **kwargs),
-        )
-
-    def _resolve_has_feature_flag(self, *_, **kwargs):
-        settings: TechnicianConfig = self._config.get_configuration(TechnicianConfig)
-        if "flag" not in kwargs:
-            return False
-        return {
-            "key": kwargs["flag"],
-            "value": FeatureFlagsSettings.get_flag_from_dict(settings.feature_flags, FeatureFlagsEnum(kwargs["flag"])),
-        }

web/package.json

@@ -1,6 +1,6 @@
 {
     "name": "web",
-    "version": "1.2.dev440",
+    "version": "1.2.dev410",
     "scripts": {
         "ng": "ng",
         "update-version": "ts-node update-version.ts",

web/src/app/models/graphql/queries.model.ts

@@ -108,15 +108,6 @@ export class Queries {
     }
   `;
 
-  static hasFeatureFlag = `
-    query HasFeatureFlag($flag: String) {
-      hasFeatureFlag(flag: $flag) {
-        key
-        value
-      }
-    }
-  `;
-
   static hasServerFeatureFlag = `
     query HasServerFeatureFlag($filter: ServerFilter, $flag: String) {
       servers(filter: $filter) {

web/src/app/modules/auth/components/login/login.component.html

@@ -4,9 +4,7 @@
       <form [formGroup]="loginForm">
         <h1>{{'auth.header' | translate}}</h1>
         <div class="input-field">
-          <input [disabled]="!basicLoginFeatureFlags" type="email" pInputText formControlName="email"
+          <input type="email" pInputText formControlName="email" placeholder="{{'common.email' | translate}}" [ngClass]="{ 'invalid-feedback-input': submitted && (
-                 placeholder="{{'common.email' | translate}}"
-                 [ngClass]="{ 'invalid-feedback-input': submitted && (
                             (loginForm.controls.email.errors && loginForm.controls.email.errors['required'] || authUserAtrErrors.email.required) ||
                             (authUserAtrErrors.email.wrongData) ||
                             (authUserAtrErrors.email.notConfirmed)
@@ -26,8 +24,7 @@
               styleClass="p-password p-component p-inputwrapper p-input-icon-right"
               Remove after update!
           -->
-          <p-password [disabled]="!basicLoginFeatureFlags" formControlName="password"
+          <p-password formControlName="password" placeholder="{{'auth.login.password' | translate}}" [ngClass]="{ 'invalid-feedback-input': submitted && (
-                      placeholder="{{'auth.login.password' | translate}}" [ngClass]="{ 'invalid-feedback-input': submitted && (
                             (loginForm.controls.password.errors && loginForm.controls.password.errors['required'] || authUserAtrErrors.password.required) ||
                             (authUserAtrErrors.password.wrongData)
                         )}" [toggleMask]="true" [feedback]="false"
@@ -42,18 +39,17 @@
         </div>
         <div class="login-form-submit">
           <button pButton label="{{'auth.login.login' | translate}}" class="btn login-form-submit-btn" (click)="login()"
-                  [disabled]="loginForm.invalid || !basicLoginFeatureFlags"></button>
+                  [disabled]="loginForm.invalid"></button>
         </div>
         <div class="login-form-sub-button-wrapper" *ngIf="!code && !state">
           <div class="login-form-sub-btn-wrapper">
-            <button pButton label="{{'auth.login.login_with_discord' | translate}}" class="btn login-form-sub-btn"
+            <button pButton label="{{'auth.login.login_with_discord' | translate}}" class="btn login-form-sub-btn" (click)="discordLogin()"></button>
-                    (click)="discordLogin()"></button>
           </div>
         </div>
         <div class="login-form-sub-button-wrapper">
           <div class="login-form-sub-btn-wrapper">
             <button pButton label="{{'auth.login.register' | translate}}" class="btn login-form-sub-btn"
-                    (click)="register()" [disabled]="!basicRegistrationFeatureFlags"></button>
+                    (click)="register()"></button>
           </div>
           <div class="login-form-sub-btn-wrapper">
             <button pButton label="{{'auth.login.forgot_password' | translate}}"

web/src/app/modules/auth/components/login/login.component.ts

@@ -13,7 +13,6 @@ import { ThemeService } from "src/app/services/theme/theme.service";
 import { throwError } from "rxjs";
 import { TranslateService } from "@ngx-translate/core";
 import { ConfirmationDialogService } from "../../../../services/confirmation-dialog/confirmation-dialog.service";
-import { GuiService } from "../../../../services/gui/gui.service";
 
 @Component({
   selector: "app-login",
@@ -33,8 +32,6 @@ export class LoginComponent implements OnInit {
   state!: string;
   user!: AuthUserDTO;
   oAuthId!: string;
-  public basicLoginFeatureFlags = false;
-  public basicRegistrationFeatureFlags = false;
 
   constructor(
     private authService: AuthService,
@@ -44,19 +41,11 @@ export class LoginComponent implements OnInit {
     private themeService: ThemeService,
     private route: ActivatedRoute,
     private confirmDialog: ConfirmationDialogService,
-    private translate: TranslateService,
+    private translate: TranslateService
-    private gui: GuiService
   ) {
   }
 
   ngOnInit(): void {
-    this.gui.hasFeatureFlag("BasicLogin").subscribe(flag => {
-      this.basicLoginFeatureFlags = flag.value;
-    });
-    this.gui.hasFeatureFlag("BasicRegistration").subscribe(flag => {
-      this.basicRegistrationFeatureFlags = flag.value;
-    });
-
     this.initLoginForm();
     this.spinnerService.showSpinner();
     if (this.authService.isLoggedIn$.value) {
@@ -118,8 +107,8 @@ export class LoginComponent implements OnInit {
 
   initLoginForm(): void {
     this.loginForm = this.formBuilder.group({
-      email: [{ value: "", disabled: !this.basicLoginFeatureFlags }, [Validators.required, Validators.email]],
+      email: ["", [Validators.required, Validators.email]],
-      password: [{ value: "", disabled: !this.basicLoginFeatureFlags }, [Validators.required, Validators.minLength(8)]]
+      password: ["", [Validators.required, Validators.minLength(8)]]
     });
   }
 

web/src/app/modules/auth/components/registration/registration.component.html

@@ -2,10 +2,9 @@
   <div class="login-form-wrapper register-form-wrapper">
     <div class="login-form">
       <form [formGroup]="loginForm">
-        <h1>{{'auth.header' | translate}}</h1>
+        <h1>sh-edraft.de</h1>
         <div class="input-field">
-          <input type="text" pInputText formControlName="firstName"
+          <input type="text" pInputText formControlName="firstName" placeholder="{{'auth.register.first_name' | translate}}"
-                 placeholder="{{'auth.register.first_name' | translate}}"
                  autocomplete="given-name">
           <div *ngIf="submitted" class="invalid-feedback">
             <div
@@ -16,8 +15,7 @@
         </div>
 
         <div class="input-field">
-          <input type="text" pInputText formControlName="lastName"
+          <input type="text" pInputText formControlName="lastName" placeholder="{{'auth.register.last_name' | translate}}"
-                 placeholder="{{'auth.register.last_name' | translate}}"
                  autocomplete="family-name">
           <div *ngIf="submitted" class="invalid-feedback">
             <div
@@ -76,14 +74,12 @@
         </div>
 
         <div class="login-form-submit">
-          <button pButton label="{{'auth.register.register' | translate}}" class="btn login-form-submit-btn"
+          <button pButton label="{{'auth.register.register' | translate}}" class="btn login-form-submit-btn" (click)="register()"
-                  (click)="register()"
                   [disabled]="loginForm.invalid"></button>
         </div>
         <div class="login-form-sub-button-wrapper">
           <div class="login-form-sub-btn-wrapper">
-            <button pButton label="{{'auth.register.login' | translate}}" class="btn login-form-sub-btn"
+            <button pButton label="{{'auth.register.login' | translate}}" class="btn login-form-sub-btn" (click)="login()"></button>
-                    (click)="login()"></button>
           </div>
         </div>
       </form>

web/src/app/modules/auth/components/registration/registration.component.ts

@@ -12,7 +12,6 @@ import { SpinnerService } from "src/app/services/spinner/spinner.service";
 import { Subject, throwError } from "rxjs";
 import { TranslateService } from "@ngx-translate/core";
 import { SettingsService } from "../../../../services/settings/settings.service";
-import { GuiService } from "../../../../services/gui/gui.service";
 
 @Component({
   selector: "app-registration",
@@ -47,8 +46,7 @@ export class RegistrationComponent implements OnInit, OnDestroy {
     private spinnerService: SpinnerService,
     private route: ActivatedRoute,
     private translate: TranslateService,
-    private settings: SettingsService,
+    private settings: SettingsService
-    private gui: GuiService
   ) {
     this.spinnerService.showSpinner();
     if (this.authService.isLoggedIn$.value) {
@@ -58,14 +56,6 @@ export class RegistrationComponent implements OnInit, OnDestroy {
   }
 
   ngOnInit(): void {
-    this.gui.hasFeatureFlag("BasicRegistration").subscribe(flag => {
-      if (flag.value) {
-        return;
-      }
-
-      this.router.navigate(["/auth/login"]);
-    });
-
     this.translate.onLangChange.pipe(takeUntil(this.unsubscriber)).subscribe(lang => {
       this.confirmPrivacyString = this.translate.instant("auth.register.confirm_privacy", { url: this.settings.getPrivacyURL() });
     });

web/src/app/services/gui/gui.service.ts

@@ -1,25 +1,24 @@
-import { HttpClient, HttpHeaders } from "@angular/common/http";
+import { HttpClient, HttpHeaders } from '@angular/common/http';
-import { Injectable } from "@angular/core";
+import { Injectable } from '@angular/core';
-import { Observable } from "rxjs";
+import { Observable, Subscribable } from 'rxjs';
-import { SettingsDTO } from "src/app/models/config/settings.dto";
+import { SettingsDTO } from 'src/app/models/config/settings.dto';
-import { SoftwareVersionDTO } from "src/app/models/config/software-version.dto";
+import { SoftwareVersionDTO } from 'src/app/models/config/software-version.dto';
-import { SettingsService } from "../settings/settings.service";
+import { SettingsService } from '../settings/settings.service';
 
 @Injectable({
-  providedIn: "root"
+  providedIn: 'root'
 })
 export class GuiService {
 
   constructor(
     private appsettings: SettingsService,
-    private http: HttpClient
+    private http: HttpClient,
-  ) {
+  ) { }
-  }
 
   getApiVersion(): Observable<SoftwareVersionDTO> {
     return this.http.get<SoftwareVersionDTO>(`${this.appsettings.getApiURL()}/api/gui/api-version`, {
       headers: new HttpHeaders({
-        "Content-Type": "application/json"
+        'Content-Type': 'application/json'
       })
     });
   }
@@ -27,7 +26,7 @@ export class GuiService {
   getSettings(): Observable<SettingsDTO> {
     return this.http.get<SettingsDTO>(`${this.appsettings.getApiURL()}/api/gui/settings`, {
       headers: new HttpHeaders({
-        "Content-Type": "application/json"
+        'Content-Type': 'application/json'
       })
     });
   }
@@ -35,18 +34,7 @@ export class GuiService {
   sendTestMail(mail: string): Observable<unknown> {
     return this.http.post(`${this.appsettings.getApiURL()}/api/gui/send-test-mail/${mail}`, {
       headers: new HttpHeaders({
-        "Content-Type": "application/json"
+        'Content-Type': 'application/json'
-      })
-    });
-  }
-
-  hasFeatureFlag(flag: string): Observable<{ key: string, value: boolean }> {
-    return this.http.get<{
-      key: string,
-      value: boolean
-    }>(`${this.appsettings.getApiURL()}/api/gui/has-feature-flag/${flag}`, {
-      headers: new HttpHeaders({
-        "Content-Type": "application/json"
       })
     });
   }

web/src/assets/version.json

@@ -2,6 +2,6 @@
     "WebVersion": {
         "Major": "1",
         "Minor": "2",
-        "Micro": "dev440"
+        "Micro": "dev410"
     }
 }