sh-edraft.de/sh_discord_bot
2
1
Fork 1
Code Issues 26 Pull Requests Actions Packages Projects 3 Releases 27 Wiki Activity

Als Nutzer möchte ich Datenänderungen nach verfolgen können #246 #248

Merged
Jonas merged 17 commits from #246 into 1.0.0 2023-03-14 18:31:37 +01:00
Conversation 5 Commits 17 Files Changed 112 +5 -5
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit efba1a4ce8 - Show all commits

10
kdb-web/src/app/modules/shared/components/history-btn/history-btn.component.ts
View File

@@ -11,15 +11,15 @@ export class HistoryBtnComponent implements OnInit {
@Input() history: History[] = [];
@Input() translationKey: string = "";
showSidebar = false;
public showSidebar: boolean = false;
edraft marked this conversation as resolved
Jonas commented 2023-03-12 14:47:28 +01:00
Review
Copy Link

Darf showSidebar keinen Typen bekommen? :(

public showSidebar: boolean = false;

Je nachdem was showSidebar tut, wäre hier vielleicht ein Store als "Single-Source-Of-Truth" sinnvoll, damit wir hier ein Observable haben.
So kann sich egal wo dieser boolische Wert anpassen und DOM-Elemente anschließened gerendert werden oder eben nicht.
Ist aber wie gesagt abhängig vom Use-Case.

Darf showSidebar keinen Typen bekommen? :( public showSidebar: boolean = false; Je nachdem was showSidebar tut, wäre hier vielleicht ein Store als "Single-Source-Of-Truth" sinnvoll, damit wir hier ein Observable haben. So kann sich egal wo dieser boolische Wert anpassen und DOM-Elemente anschließened gerendert werden oder eben nicht. Ist aber wie gesagt abhängig vom Use-Case.
edraft commented 2023-03-12 18:45:27 +01:00
Review
Copy Link

Generell hast du recht. Wäre für das was die Komponente macht jedoch OP.

Generell hast du recht. Wäre für das was die Komponente macht jedoch OP.
constructor() {
public constructor() {
}
ngOnInit(): void {
public ngOnInit(): void {
}
openHistory(): void {
public openHistory(): void {
this.showSidebar = true;
let oldHistory: Partial<History> = {};
for (const history of this.history) {
@@ -36,7 +36,7 @@ export class HistoryBtnComponent implements OnInit {
}
}
getAttributeTranslationKey(key: string) {
public getAttributeTranslationKey(key: string): string {
return `common.history.${key}`;
}