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
Owner
#246
edraft self-assigned this 2023-03-08 14:58:27 +01:00
edraft added 12 commits 2023-03-08 14:58:27 +01:00
edraft requested review from Ebola-Chan 2023-03-08 14:58:31 +01:00
edraft requested review from Jonas 2023-03-08 14:58:31 +01:00
edraft added this to the Sprint 1.0.0 project 2023-03-08 14:58:36 +01:00
edraft changed title from WIP: Als Nutzer möchte ich Datenänderungen nach verfolgen können #246 to Als Nutzer möchte ich Datenänderungen nach verfolgen können #246 2023-03-11 01:12:48 +01:00
edraft force-pushed #246 from 928c099d01 to ee503f76f2 2023-03-11 01:19:23 +01:00 Compare
edraft added 3 commits 2023-03-11 03:10:12 +01:00
edraft added 1 commit 2023-03-12 04:55:12 +01:00
Jonas reviewed 2023-03-12 14:42:56 +01:00
@ -0,0 +20,4 @@
it('should create', () => {
expect(component).toBeTruthy();
});
});
Contributor

Tests schreiben üben wir aber nochmal ^^

Tests schreiben üben wir aber nochmal ^^
edraft marked this conversation as resolved
Jonas requested changes 2023-03-12 14:49:49 +01:00
@ -0,0 +11,4 @@
@Input() history: History[] = [];
@Input() translationKey: string = "";
showSidebar = false;
Contributor

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.
Author
Owner

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.
edraft marked this conversation as resolved
@ -0,0 +40,4 @@
return `common.history.${key}`;
}
}
Contributor

Mir fehlen sämtliche Sichtbarkeitsregeln:

  • public constructor..
  • public ngOnInit()...
  • ...
Mir fehlen sämtliche Sichtbarkeitsregeln: - public constructor.. - public ngOnInit()... - ...
Jonas marked this conversation as resolved
@ -160,6 +160,7 @@
<td>
<div class="btn-wrapper">
<app-history-btn [history]="autoRoleRule.history" translationKey="view.server.auto_roles.rules.header"></app-history-btn>
Contributor

"App" als Präfix finde ich immer hässlich irgendwie.
Bin aber vielleicht auch nur ich, lol.

"App" als Präfix finde ich immer hässlich irgendwie. Bin aber vielleicht auch nur ich, lol.
Author
Owner

ja bist du xD

ja bist du xD
edraft marked this conversation as resolved
edraft added 1 commit 2023-03-12 18:46:48 +01:00
Ebola-Chan requested changes 2023-03-14 16:21:39 +01:00
@ -0,0 +40,4 @@
@ServiceProviderABC.inject
def icon_url(self, bot: DiscordBotServiceABC) -> str:
guild = bot.get_guild(self.discord_id)
return None if guild is None else guild.icon.url
Contributor

Gibt es Fälle, bei dem icon null ist?

Gibt es Fälle, bei dem ```icon``` null ist?
Contributor

Wenn nicht, kann dies ingnoriert werden.

Wenn nicht, kann dies ingnoriert werden.
Author
Owner

nope

nope
edraft marked this conversation as resolved
@ -136,2 +156,3 @@
"of": "von",
"reset_filters": "Filter zurücksetzen"
"reset_filters": "Filter zurücksetzen",
"history": {
Contributor

Zeile 131 existier bereits ein Objekt namens "history"

Zeile 131 existier bereits ein Objekt namens "history"
Contributor

Also in Zeile 131 und 158 gibt es das Objekt history

Also in Zeile 131 und 158 gibt es das Objekt ```history```
Ebola-Chan marked this conversation as resolved
edraft added 1 commit 2023-03-14 17:01:05 +01:00
Ebola-Chan approved these changes 2023-03-14 17:02:30 +01:00
Jonas approved these changes 2023-03-14 18:31:23 +01:00
Jonas merged commit 283eaabef6 into 1.0.0 2023-03-14 18:31:37 +01:00
Jonas deleted branch #246 2023-03-14 18:31:37 +01:00
Sign in to join this conversation.
No description provided.