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 +2518 -214
edraft commented 2023-03-08 14:58:27 +01:00
Owner
Copy Link

#246

#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
Set version for rc 18fe6ecb86
Build version 505c87b996
Build frontend 4d0e8898fe
Removed minecraft_id 883811d156
Fixed auto-role error 23e17c1f38
Updated docker compose bddcd3929a
Fixed time calculation for user joined gs/vs 1664c67763
Fixed time calculation for user joined gs/vs 3324cf3ac3
Fixed frontend permissions a701807831
Reload levelsystem after mutate levels #245 22d26bf032
Put boot log send message in loop task #243 d1420d18c0
Retranslate sidebar on lang change 21f0f32322
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
Improved history component #246 68026e4b47
Fixed mysql scripts #246 e3c47ce5b1
Fixed mysql scripts #246 ad00dce5d9
edraft added 1 commit 2023-03-12 04:55:12 +01:00
Fixed db set date & show only changed fields in history view #246 b1a0115e8b
Jonas reviewed 2023-03-12 14:42:56 +01:00
kdb-web/src/app/modules/shared/components/history-btn/history-btn.component.spec.ts
@@ -0,0 +20,4 @@
it('should create', () => {
expect(component).toBeTruthy();
});
});
Jonas commented 2023-03-12 14:42:56 +01:00
Contributor
Copy Link

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
kdb-web/src/app/modules/shared/components/history-btn/history-btn.component.ts
@@ -0,0 +11,4 @@
@Input() history: History[] = [];
@Input() translationKey: string = "";
showSidebar = false;
Jonas commented 2023-03-12 14:47:28 +01:00
Contributor
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
Author
Owner
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.
edraft marked this conversation as resolved
kdb-web/src/app/modules/shared/components/history-btn/history-btn.component.ts
@@ -0,0 +40,4 @@
return `common.history.${key}`;
}
}
Jonas commented 2023-03-12 14:44:48 +01:00
Contributor
Copy Link

Mir fehlen sämtliche Sichtbarkeitsregeln:

  • public constructor..
  • public ngOnInit()...
  • ...
Mir fehlen sämtliche Sichtbarkeitsregeln: - public constructor.. - public ngOnInit()... - ...
Jonas marked this conversation as resolved
kdb-web/src/app/modules/view/server/auto-role/components/auto-roles-rules/auto-roles-rules.component.html
@@ -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>
Jonas commented 2023-03-12 14:48:27 +01:00
Contributor
Copy Link

"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.
edraft commented 2023-03-12 18:46:20 +01:00
Author
Owner
Copy Link

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
Added type hints #246 efba1a4ce8
Ebola-Chan requested changes 2023-03-14 16:21:39 +01:00
kdb-bot/src/bot_data/model/server_history.py
@@ -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
Ebola-Chan commented 2023-03-14 16:15:05 +01:00
Contributor
Copy Link

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

Gibt es Fälle, bei dem ```icon``` null ist?
Ebola-Chan commented 2023-03-14 16:22:32 +01:00
Contributor
Copy Link

Wenn nicht, kann dies ingnoriert werden.

Wenn nicht, kann dies ingnoriert werden.
edraft commented 2023-03-14 16:57:48 +01:00
Author
Owner
Copy Link

nope

nope
edraft marked this conversation as resolved
kdb-web/src/assets/i18n/de.json
@@ -136,2 +156,3 @@
"of": "von",
"reset_filters": "Filter zurücksetzen"
"reset_filters": "Filter zurücksetzen",
"history": {
Ebola-Chan commented 2023-03-12 20:58:46 +01:00
Contributor
Copy Link

Zeile 131 existier bereits ein Objekt namens "history"

Zeile 131 existier bereits ein Objekt namens "history"
Ebola-Chan commented 2023-03-14 16:23:19 +01:00
Contributor
Copy Link

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
Fixed translation #246 b7e72888f7
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.
Reviewers
No Reviewers
Ebola-Chan
Jonas
Labels
Clear labels
MVP
web-interface
bug
Something is not working
bugfix
Solution of a problem
cli
Changes for command line interface
discussion
Something needs to be discussed
documentation
Indicates a need for improvements or additions to documentation
duplicate
This issue or pull request already exists
expense
high
expense
low
expense
medium
expense
very-high
help wanted
Need some help
improvement
Improvement of a feature
invalid
Something is wrong
priority
blocker
priority
high
priority
low
priority
medium
question
More information is needed
refactoring
Code need some refactoring
release
Create a release
story
New feature
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project Sprint 1.0.0
Assignees
Clear assignees
ci edraft edraft-dev
No Assignees edraft
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sh-edraft.de/sh_discord_bot#248
No description provided.
Delete Branch "#246"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?