CreateGameView erstellen #67

Merged
sneeex merged 35 commits from feature/3-creategameview-erstellen into development 2025-12-10 20:43:54 +00:00
Owner

CreateGameView erstellen

Zugehörige Issue(s):
Closes #3 #87

Beschreibung

Es wurde eine Ansicht zum Erstellen von Spielen hinzugefügt, bei welcher der Spielname, das Regelset, die Gruppe wie auch einzelne Spieler gesetzt werden können.

Änderungen

  • CreateGameView erstellt, welche es ermöglich ein Spiel mit allen Eigentschaften zu erstellen und nach dem erstellen direkt zur GameResultView navigiert.

ChooseGroupView implementiert

  • In der ChooseGroupView kann eine Gruppe ausgewählt werden.
  • Die vorher ausgewählte Gruppe wird hervorgehoben.
  • Gruppen können auch deselected werden.
  • Spieler die Teil der ausgewählten Gruppe sind, tauchen nicht mehr in der Player Selection auf

ChooseGameView implementiert

  • In der ChooseGameView kann eine Game ausgewählt werden.
  • Das vorher ausgewählte Game wird farbig hervorgehoben.
  • Games können auch deselected werden.
  • Wenn ein Game ausgewählt wird, wird automatisch auch Ruleset gesetzt.
  • Aktuell sind hier nur Beispiele drin. Für die Zukunft wird noch eine Möglichkeit gebraucht, Games zu erstellen.

ChooseRulesetView implementiert

  • In der ChooseRulesetView kann eine Ruleset ausgewählt werden.
  • Das vorher ausgewählte Ruleset wird hervorgehoben.
  • Rulesets können auch deselected werden.
  • Wenn ein Ruleset ausgewählt wird, setzt sich das Game automatisch zurück.
  • Aktuell sind hier nur Platzhalter für die späteren main Rulesets drin.
# CreateGameView erstellen **Zugehörige Issue(s):** Closes #3 #87 ## Beschreibung Es wurde eine Ansicht zum Erstellen von Spielen hinzugefügt, bei welcher der Spielname, das Regelset, die Gruppe wie auch einzelne Spieler gesetzt werden können. ## Änderungen - `CreateGameView` erstellt, welche es ermöglich ein Spiel mit allen Eigentschaften zu erstellen und nach dem erstellen direkt zur `GameResultView` navigiert. ### `ChooseGroupView` implementiert - In der `ChooseGroupView` kann eine Gruppe ausgewählt werden. - Die vorher ausgewählte Gruppe wird hervorgehoben. - Gruppen können auch deselected werden. - Spieler die Teil der ausgewählten Gruppe sind, tauchen nicht mehr in der Player Selection auf ### `ChooseGameView` implementiert - In der `ChooseGameView` kann eine Game ausgewählt werden. - Das vorher ausgewählte Game wird farbig hervorgehoben. - Games können auch deselected werden. - Wenn ein Game ausgewählt wird, wird automatisch auch Ruleset gesetzt. - Aktuell sind hier nur Beispiele drin. Für die Zukunft wird noch eine Möglichkeit gebraucht, Games zu erstellen. ### `ChooseRulesetView` implementiert - In der `ChooseRulesetView` kann eine Ruleset ausgewählt werden. - Das vorher ausgewählte Ruleset wird hervorgehoben. - Rulesets können auch deselected werden. - Wenn ein Ruleset ausgewählt wird, setzt sich das Game automatisch zurück. - Aktuell sind hier nur Platzhalter für die späteren main Rulesets drin.
flixcoo added this to the Minimum Viable Product (MVP) milestone 2025-11-23 21:37:41 +00:00
flixcoo self-assigned this 2025-11-23 21:37:41 +00:00
flixcoo added 2 commits 2025-11-23 21:37:41 +00:00
Implemented View for choosing group and ruleset
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 2m13s
Pull Request Pipeline / test (pull_request) Successful in 2m15s
e71cb11295
flixcoo changed title from CreateGameView erstellen to WIP: CreateGameView erstellen 2025-11-23 21:37:45 +00:00
flixcoo added 5 commits 2025-11-24 12:53:35 +00:00
flixcoo added 5 commits 2025-11-24 15:23:33 +00:00
moved input widgets to new folder
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
9efbc12909
flixcoo added 4 commits 2025-11-24 22:06:41 +00:00
# Conflicts:
#	lib/presentation/views/main_menu/create_group_view.dart
Implemented app skeleton
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m3s
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
5fbf2ccb45
flixcoo added 1 commit 2025-11-25 21:39:01 +00:00
Implemented player selection
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Pull Request Pipeline / lint (pull_request) Successful in 2m12s
2838376434
flixcoo requested review from Owners 2025-11-25 21:41:10 +00:00
flixcoo added the
PR
Ready for Review
2
label 2025-11-25 21:41:14 +00:00
Owner

wollen wir nicht mit mergen warten bis #20 gemergt ist?

wollen wir nicht mit mergen warten bis #20 gemergt ist?
sneeex added a new dependency 2025-11-25 21:55:10 +00:00
Author
Owner

wollen wir nicht mit mergen warten bis #20 gemergt ist?

ja

> wollen wir nicht mit mergen warten bis #20 gemergt ist? ja
sneeex requested changes 2025-11-25 22:19:34 +00:00
Dismissed
@@ -11,0 +16,4 @@
static BoxDecoration highlightedBoxDecoration = BoxDecoration(
color: boxColor,
border: Border.all(color: Colors.blue),
Owner

zu primary color oder farbe ähnlich primary color ändern

zu primary color oder farbe ähnlich primary color ändern
flixcoo marked this conversation as resolved
@@ -11,0 +18,4 @@
color: boxColor,
border: Border.all(color: Colors.blue),
borderRadius: BorderRadius.circular(12),
boxShadow: [BoxShadow(color: Colors.blue.withAlpha(120), blurRadius: 12)],
Owner

zu primary color oder farbe ähnlich primary color ändern

zu primary color oder farbe ähnlich primary color ändern
flixcoo marked this conversation as resolved
@@ -0,0 +36,4 @@
backgroundColor: CustomTheme.backgroundColor,
scrolledUnderElevation: 0,
title: const Text(
'Choose Gametype',
Owner

Macht keinen Sinn, da gametype und rulesets ausgewählt werden können

Macht keinen Sinn, da gametype und rulesets ausgewählt werden können
Author
Owner

Habs jetzt zu Choose Ruleset geändert, da man mit den Gametypes ja auch im Prinzip nur n Ruleset festlegt

Habs jetzt zu `Choose Ruleset` geändert, da man mit den Gametypes ja auch im Prinzip nur n Ruleset festlegt
sneeex marked this conversation as resolved
@@ -0,0 +48,4 @@
padding: const EdgeInsets.symmetric(horizontal: 12, vertical: 8),
child: TabBar(
padding: const EdgeInsets.symmetric(horizontal: 5),
// Label Settings
Owner

warum so viele comments? ist doch klar wofür die settings jeweils sind, also label indicator, divider etc

warum so viele comments? ist doch klar wofür die settings jeweils sind, also label indicator, divider etc
Author
Owner

Wollte das einfach bisschen klarer aufteilen

Wollte das einfach bisschen klarer aufteilen
sneeex marked this conversation as resolved
@@ -0,0 +105,4 @@
),
const Center(
child: Text(
'No gametypes available',
Owner

auch hier no gametypes und nicht irgendwas anderes bzw gametypes or rulesets

auch hier no gametypes und nicht irgendwas anderes bzw gametypes or rulesets
Author
Owner

Was meinst du?

Was meinst du?
Owner

Da steht noch gametypes obwohls ruleset ist

Da steht noch gametypes obwohls ruleset ist
Author
Owner

Ja weil da die gametypes hin sollen, also sowas wie "Cabo", "Uno", etc.

Ja weil da die gametypes hin sollen, also sowas wie "Cabo", "Uno", etc.
Owner

Hä aber das sind doch rulesets und nicht gametypes?? Ist ja das file choose_ruleset_view

Hä aber das sind doch rulesets und nicht gametypes?? Ist ja das file choose_ruleset_view
Author
Owner

hätte ich das choose_ruleset_or_gametypes_view.dart nennen sollen????

hätte ich das `choose_ruleset_or_gametypes_view.dart` nennen sollen????
Owner

evtl besser gametype und ruleset aufteilen und wenn man dann einen gametype auswählt, wird automatisch ein ruleset ausgewählt, ein gametype ist aber nicht erforderlich, ein ruleset schon. Das ruleset lässt sich aber nur ändern wenn kein gametype ausgewählt ist, bzw. wenn das ruleset geändert wird und ein gametype ausgewählt ist, wird der gametype auf null gesetzt.

evtl besser gametype und ruleset aufteilen und wenn man dann einen gametype auswählt, wird automatisch ein ruleset ausgewählt, ein gametype ist aber nicht erforderlich, ein ruleset schon. Das ruleset lässt sich aber nur ändern wenn kein gametype ausgewählt ist, bzw. wenn das ruleset geändert wird und ein gametype ausgewählt ist, wird der gametype auf null gesetzt.
Owner

@gelbeinhalb deine Meinung? So wie jetzt oder wie ich vorgeschlagen habe? Felix checkt beides

@gelbeinhalb deine Meinung? So wie jetzt oder wie ich vorgeschlagen habe? Felix checkt beides
flixcoo marked this conversation as resolved
@@ -0,0 +42,4 @@
Group? selectedGroup;
/// The index of the currently selected group in [groupsList] to mark it in
/// the [ChooseGroupView]
Owner

warum auch hier bei allen vars comments??

warum auch hier bei allen vars comments??
Author
Owner

Damit halt jede Variable klar ist und man auch entsprechende Docs hat, wenn man darüber hovert

Damit halt jede Variable klar ist und man auch entsprechende Docs hat, wenn man darüber hovert
sneeex marked this conversation as resolved
@@ -0,0 +181,4 @@
(p) => !selectedGroup!.members.any((m) => m.id == p.id),
)
.toList(),
);
Owner

mehrere prints noch im code

mehrere prints noch im code
flixcoo marked this conversation as resolved
@@ -0,0 +221,4 @@
)
.toList(),
onChanged: (value) {
print(value);
Owner

auch hier print

auch hier print
flixcoo marked this conversation as resolved
@@ -0,0 +236,4 @@
selectedRuleset == null)
? null
: () async {
Game game = Game(
Owner
grafik.png game lässt sich nicht erstellen, obwohl spieler, game name und ruleset ausgewählt ist (funktioniert auch mit 2 spielern nicht, hier muss aber darauf geachtet werden, dass ein spiel nur mit min. 2 spielern erstellt werden kann)
<img width="318" alt="grafik.png" src="attachments/1bb19145-834c-4a72-9399-a43972f77e46"> game lässt sich nicht erstellen, obwohl spieler, game name und ruleset ausgewählt ist (funktioniert auch mit 2 spielern nicht, hier muss aber darauf geachtet werden, dass ein spiel nur mit min. 2 spielern erstellt werden kann)
132 KiB
Author
Owner

Ja ups haha ich hab ganz vergessen irgendwas mit den ausgewählten Spielern zu machen, hab die einfach nur geprinted haha. Hab die jetzt auch included und sollte jetzt funktionieren

Ja ups haha ich hab ganz vergessen irgendwas mit den ausgewählten Spielern zu machen, hab die einfach nur geprinted haha. Hab die jetzt auch included und sollte jetzt funktionieren
sneeex marked this conversation as resolved
@@ -173,0 +183,4 @@
child: Center(
child: CustomWidthButton(
text: 'Create Game',
sizeRelativeToWidth: 0.95,
Owner

0.9, 0.95 sieht komisch aus mit der navbar, soll so sein wie im groupsview

0.9, 0.95 sieht komisch aus mit der navbar, soll so sein wie im groupsview
Author
Owner

Also ja ich änder das, aber den Button implementiert ja eh yannick in #2 , deswegen war der eh nur mittel zum swag

Also ja ich änder das, aber den Button implementiert ja eh yannick in #2 , deswegen war der eh nur mittel zum swag
sneeex marked this conversation as resolved
flixcoo added 5 commits 2025-11-26 11:40:11 +00:00
Changed create game button size
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
b5234c765c
flixcoo requested review from sneeex 2025-11-26 11:40:52 +00:00
flixcoo changed title from WIP: CreateGameView erstellen to CreateGameView erstellen 2025-11-26 11:40:59 +00:00
flixcoo added
PR
In Review
1
and removed
PR
Ready for Review
2
labels 2025-11-26 11:41:57 +00:00
flixcoo added 1 commit 2025-11-26 12:12:19 +00:00
Implemented ChooseTile
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m2s
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
745aaef978
sneeex requested changes 2025-11-26 12:40:47 +00:00
Dismissed
sneeex left a comment
Owner

Und ruleset_list_tile bitte umbenennen zu was generellerem, nicht auf ruleset bezogen

Und `ruleset_list_tile` bitte umbenennen zu was generellerem, nicht auf ruleset bezogen
flixcoo added 5 commits 2025-11-26 13:44:46 +00:00
flixcoo added 1 commit 2025-11-28 13:07:47 +00:00
Renamed widget
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
Pull Request Pipeline / test (pull_request) Successful in 2m8s
94b113eb95
flixcoo added 1 commit 2025-11-28 13:41:55 +00:00
Implemented choose game view
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m3s
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
236a737fd1
flixcoo added 3 commits 2025-11-28 22:15:16 +00:00
Added conditional description
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Pull Request Pipeline / lint (pull_request) Successful in 2m4s
b4ccb567b5
flixcoo changed title from CreateGameView erstellen to WIP: CreateGameView erstellen 2025-11-30 15:39:45 +00:00
flixcoo removed the
PR
In Review
1
label 2025-11-30 15:39:52 +00:00
flixcoo added a new dependency 2025-12-02 15:41:15 +00:00
flixcoo added 1 commit 2025-12-06 15:54:44 +00:00
Merge branch 'development' into feature/3-creategameview-erstellen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m8s
Pull Request Pipeline / lint (pull_request) Successful in 2m10s
527f163346
flixcoo added 2 commits 2025-12-06 16:13:38 +00:00
Updated choosing mechanism
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
10aad47124
flixcoo added 2 commits 2025-12-06 16:24:39 +00:00
Adjusted margin
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
a1a995777b
flixcoo requested review from gelbeinhalb 2025-12-06 16:24:44 +00:00
flixcoo requested review from sneeex 2025-12-06 16:24:46 +00:00
flixcoo added the
PR
Ready for Review
2
label 2025-12-06 16:24:51 +00:00
flixcoo changed title from WIP: CreateGameView erstellen to CreateGameView erstellen 2025-12-06 16:24:54 +00:00
flixcoo added a new dependency 2025-12-06 16:26:30 +00:00
flixcoo added 1 commit 2025-12-06 16:42:15 +00:00
Implemented final navigation to GameResultView
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m8s
Pull Request Pipeline / lint (pull_request) Successful in 2m19s
6ae0471fa2
flixcoo added 1 commit 2025-12-06 22:19:25 +00:00
Merge branch 'development' into feature/3-creategameview-erstellen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
4fc1959704
flixcoo added 1 commit 2025-12-07 14:48:54 +00:00
Merge branch 'development' into feature/3-creategameview-erstellen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m10s
Pull Request Pipeline / lint (pull_request) Successful in 2m20s
1044ee9bea
sneeex requested changes 2025-12-07 17:40:56 +00:00
Dismissed
@@ -0,0 +56,4 @@
child: CustomSearchBar(
controller: searchBarController,
hintText: 'Game Name',
),
Owner

Es ist kein OnChanged implementiert, die search functionality ist insgesamt nicht gegeben hier.

Note: Das onChanged sollte aber vllt. auch required sein in der CustomSearchBar (weiß gerade nicht ob ich das aus nem grund optional gelassen habe, macht aber ja glaube keinen sinn oder?)

Es ist kein OnChanged implementiert, die search functionality ist insgesamt nicht gegeben hier. Note: Das onChanged sollte aber vllt. auch required sein in der CustomSearchBar (weiß gerade nicht ob ich das aus nem grund optional gelassen habe, macht aber ja glaube keinen sinn oder?)
Author
Owner

Also ich habs explizit weggelassen, weil ich der einfachhalt halber das aktuell ausgesuchte Spiel über einen Index markiere. Wenn ich das jetzt implementieren würde, müsste ich dass unnötig kompliziert implementieren, weil die Games am Ende eh eine ID haben über die ich das markierte Identifizieren kann, deswegen würd ich das an dieser Stelle weglassen

Also ich habs explizit weggelassen, weil ich der einfachhalt halber das aktuell ausgesuchte Spiel über einen Index markiere. Wenn ich das jetzt implementieren würde, müsste ich dass unnötig kompliziert implementieren, weil die Games am Ende eh eine ID haben über die ich das markierte Identifizieren kann, deswegen würd ich das an dieser Stelle weglassen
Owner

dann mach die searchbar weg, entweder searchbar und functionality oder nicht, aber nicht so halbe sachen.

dann mach die searchbar weg, entweder searchbar und functionality oder nicht, aber nicht so halbe sachen.
Author
Owner

Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen

Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen
Owner

Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen

dann mach die functionality rein, entweder oder.
Du hast ja nur kein bock

> Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen dann mach die functionality rein, entweder oder. Du hast ja nur kein bock
flixcoo marked this conversation as resolved
@@ -0,0 +71,4 @@
onPressed: () async {
setState(() {
selectedGameIndex = index;
});
Owner

warum ist unselecten hier nicht implementiert, beim choose group view aber schon?

warum ist unselecten hier nicht implementiert, beim choose group view aber schon?
Author
Owner

ups, vergessen

ups, vergessen
flixcoo marked this conversation as resolved
@@ -46,12 +56,11 @@ class _ChooseGroupViewState extends State<ChooseGroupView> {
return GestureDetector(
Owner

Warum lässt sich in Choose Game View suchen, im Choose Group View aber nicht? Können beides ja potenziell sehr lange Listen sein

Warum lässt sich in Choose Game View suchen, im Choose Group View aber nicht? Können beides ja potenziell sehr lange Listen sein
Author
Owner

Erledigt (So würde dass dann auch für die Games funktionieren)

Erledigt (So würde dass dann auch für die Games funktionieren)
flixcoo marked this conversation as resolved
@@ -79,3 +84,4 @@
Ruleset.leastPoints,
'Least Points',
'Inverse scoring: the player with the fewest points wins.',
),
Owner

Du hast doch in enums die func translateRulesetToString für die Namen, warum nutzt du nicht die einfach für die Namen hier?

Du hast doch in enums die func translateRulesetToString für die Namen, warum nutzt du nicht die einfach für die Namen hier?
Author
Owner

Hab ich danach erst implementiert, erledigt.

Hab ich danach erst implementiert, erledigt.
flixcoo marked this conversation as resolved
@@ -122,6 +134,27 @@ class _CreateGameViewState extends State<CreateGameView> {
},
Owner

die Function für onChanged wird nicht gebraucht, weil ja schon ein Listener auf den TextEditingController gesetzt ist

die Function für onChanged wird nicht gebraucht, weil ja schon ein Listener auf den TextEditingController gesetzt ist
flixcoo marked this conversation as resolved
@@ -200,0 +230,4 @@
final db = Provider.of<AppDatabase>(
context,
listen: false,
);
Owner

db ist doch schon assignt? warum nochmal? siehe zeile 99

db ist doch schon assignt? warum nochmal? siehe zeile 99
flixcoo marked this conversation as resolved
@@ -223,4 +253,3 @@
/// Determines whether the "Create Game" button should be enabled based on
/// the current state of the input fields.
bool _enableCreateGameButton() {
Owner

hier sicherstellen, dass mindestens 2 spieler in nem game sind oder nicht?
kann leider nicht an der richtigen stelle kommentieren, aber in der function halt

hier sicherstellen, dass mindestens 2 spieler in nem game sind oder nicht? kann leider nicht an der richtigen stelle kommentieren, aber in der function halt
flixcoo marked this conversation as resolved
@@ -0,0 +44,4 @@
style: const TextStyle(
fontWeight: FontWeight.bold,
fontSize: 18,
),
Owner

title ist nicht overflow sicher und verdrängt das badge

grafik.png

title ist nicht overflow sicher und verdrängt das badge ![grafik.png](/attachments/bcfc7b02-e2a3-483f-ba90-d01bdf33a3b5)
flixcoo marked this conversation as resolved
@@ -0,0 +64,4 @@
color: Colors.white,
fontSize: 12,
fontWeight: FontWeight.bold,
),
Owner

badgetext nicht overflow sicher
grafik.png

badgetext nicht overflow sicher ![grafik.png](/attachments/44ff847b-617f-4a67-9a09-1547fe967e1c)
flixcoo marked this conversation as resolved
flixcoo added 8 commits 2025-12-07 18:31:23 +00:00
flixcoo requested review from sneeex 2025-12-07 18:31:25 +00:00
flixcoo added
PR
In Review
1
and removed
PR
Ready for Review
2
labels 2025-12-07 18:31:29 +00:00
sneeex requested changes 2025-12-07 19:05:57 +00:00
Dismissed
@@ -63,0 +88,4 @@
child: GroupTile(
group: filteredGroups[index],
isHighlighted: selectedGroupId == filteredGroups[index].id,
),
Owner

hier noch etwas einblenden wenn kein suchergebnis da ist, also filteredGroups leer ist (bspw. topcenteredmessage)

hier noch etwas einblenden wenn kein suchergebnis da ist, also filteredGroups leer ist (bspw. topcenteredmessage)
flixcoo marked this conversation as resolved
sneeex requested changes 2025-12-07 19:11:03 +00:00
Dismissed
sneeex left a comment
Owner

Komisches Verhalten bei Group deselect, auch im Zusammenhang mit Ruleset:
https://drive.google.com/file/d/1ExIUfSBy3nRWqro8bU-K3G7YVpGaYp-Y/view?usp=sharing

Ruleset und Group alleine gehen aber.

Komisches Verhalten bei Group deselect, auch im Zusammenhang mit Ruleset: https://drive.google.com/file/d/1ExIUfSBy3nRWqro8bU-K3G7YVpGaYp-Y/view?usp=sharing Ruleset und Group alleine gehen aber.
Owner

Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt?

Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt?
Author
Owner

Komisches Verhalten bei Group deselect, auch im Zusammenhang mit Ruleset:
https://drive.google.com/file/d/1ExIUfSBy3nRWqro8bU-K3G7YVpGaYp-Y/view?usp=sharing

Ruleset und Group alleine gehen aber.

Also dass das ruleset weggeht, wenn du auch das game deselectest soll ja auch so, das ruleset ist in dem moment ja an das game geknüpft. Das andere schau ich mir an

> Komisches Verhalten bei Group deselect, auch im Zusammenhang mit Ruleset: > https://drive.google.com/file/d/1ExIUfSBy3nRWqro8bU-K3G7YVpGaYp-Y/view?usp=sharing > > Ruleset und Group alleine gehen aber. Also dass das ruleset weggeht, wenn du auch das game deselectest soll ja auch so, das ruleset ist in dem moment ja an das game geknüpft. Das andere schau ich mir an
Author
Owner

Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt?

Also eigentlich nicht, aber macht ja theoretisch sinn, weil in der gruppe ja theoretisch die spieler sein können, die du schon ausgewählt hattest. Oder wie würdest du das machen?

> Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt? Also eigentlich nicht, aber macht ja theoretisch sinn, weil in der gruppe ja theoretisch die spieler sein können, die du schon ausgewählt hattest. Oder wie würdest du das machen?
Owner

Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt?

Also eigentlich nicht, aber macht ja theoretisch sinn, weil in der gruppe ja theoretisch die spieler sein können, die du schon ausgewählt hattest. Oder wie würdest du das machen?

Die Spieler die dann doppelt sind aus selectedPlayers entfernen und die anderen da behalten

> > Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt? > > Also eigentlich nicht, aber macht ja theoretisch sinn, weil in der gruppe ja theoretisch die spieler sein können, die du schon ausgewählt hattest. Oder wie würdest du das machen? Die Spieler die dann doppelt sind aus selectedPlayers entfernen und die anderen da behalten
flixcoo added 2 commits 2025-12-07 21:15:24 +00:00
Implemented leaving players in the player selection and filtering those who are in the group
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m11s
Pull Request Pipeline / lint (pull_request) Successful in 2m12s
708157df54
Author
Owner

Also eigentlich nicht, aber macht ja theoretisch sinn, weil in der gruppe ja theoretisch die spieler sein können, die du schon ausgewählt hattest. Oder wie würdest du das machen?

Habe das jetzt mal implementiert, so dass die Spieler 1. bleiben und 2. alle Spieler welche schon teil der Gruppe sind rausfliegen

> Also eigentlich nicht, aber macht ja theoretisch sinn, weil in der gruppe ja theoretisch die spieler sein können, die du schon ausgewählt hattest. Oder wie würdest du das machen? Habe das jetzt mal implementiert, so dass die Spieler 1. bleiben und 2. alle Spieler welche schon teil der Gruppe sind rausfliegen
flixcoo requested review from sneeex 2025-12-07 21:16:11 +00:00
flixcoo added 1 commit 2025-12-07 21:23:47 +00:00
Added empty message
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m14s
Pull Request Pipeline / lint (pull_request) Successful in 2m14s
7cff48ebc0
Owner

wenn das hier gemerged ist machen wir zuerst #78 bevor irgendwas anderes angefangen wird oder?

wenn das hier gemerged ist machen wir zuerst #78 bevor irgendwas anderes angefangen wird oder?
Author
Owner

wenn das hier gemerged ist machen wir zuerst #78 bevor irgendwas anderes angefangen wird oder?

ja

> wenn das hier gemerged ist machen wir zuerst #78 bevor irgendwas anderes angefangen wird oder? ja
sneeex requested changes 2025-12-08 13:39:55 +00:00
Dismissed
@@ -63,0 +78,4 @@
icon: Icons.info,
title: 'No group',
message: 'There is no group matching your search.',
),
Owner

Hier bitte ohne Punkt, das hatten wir ja gesagt, und wir müssen uns darauf einigen ob der Titel infos erhält, oder nur keywords wie info/error, weil ich das bis jetzt so gemacht habe.
Sonst passt so

Hier bitte ohne Punkt, das hatten wir ja gesagt, und wir müssen uns darauf einigen ob der Titel infos erhält, oder nur keywords wie info/error, weil ich das bis jetzt so gemacht habe. Sonst passt so
Author
Owner

Wenn der Titel keine Infos enthalten soll, würd ich daraus n enum Attribut machen damit der je nach enum gesetzt wird (Icon und Titel)

Wenn der Titel keine Infos enthalten soll, würd ich daraus n enum Attribut machen damit der je nach enum gesetzt wird (Icon und Titel)
Owner

ja findste mit oder ohne richtigen Title besser?

ja findste mit oder ohne richtigen Title besser?
Owner

ist mir auch egal, lass es so oder nicht. Wenn du es mit Info/Error im Titel willst, mach am besten das einmal temporär in den Titel und dann nen Issue für Änderung der anderen Titel in eine enum Version oder halt eine Version mit aussagekräftigem Titel.

Aber auf jeden Fall den Punkt weg machen am Ende von Message
dann approve ich

ist mir auch egal, lass es so oder nicht. Wenn du es mit Info/Error im Titel willst, mach am besten das einmal temporär in den Titel und dann nen Issue für Änderung der anderen Titel in eine enum Version oder halt eine Version mit aussagekräftigem Titel. **Aber auf jeden Fall den Punkt weg machen am Ende von Message** dann approve ich
Author
Owner

erledigt

erledigt
flixcoo marked this conversation as resolved
Owner

Obwohl andere Sache: gehört #87 nicht auch hierzu? Hat ja wahrscheinlich damit zu tun, dass du nach dem Aufrufen vom Navigator mit dem du die CreateGameView zeigst nicht reloadest

Obwohl andere Sache: gehört #87 nicht auch hierzu? Hat ja wahrscheinlich damit zu tun, dass du nach dem Aufrufen vom Navigator mit dem du die CreateGameView zeigst nicht reloadest
gelbeinhalb approved these changes 2025-12-09 08:23:02 +00:00
Dismissed
Owner

Nicht mergen

Nicht mergen
Owner

Nicht mergen

ja du musst ja auch noch approven

> Nicht mergen ja du musst ja auch noch approven
flixcoo added 2 commits 2025-12-09 17:26:11 +00:00
Fixed state problem with games list
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
701500c7e2
flixcoo dismissed gelbeinhalb's review 2025-12-09 17:26:11 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

Author
Owner

Obwohl andere Sache: gehört #87 nicht auch hierzu? Hat ja wahrscheinlich damit zu tun, dass du nach dem Aufrufen vom Navigator mit dem du die CreateGameView zeigst nicht reloadest

Fixed

> Obwohl andere Sache: gehört #87 nicht auch hierzu? Hat ja wahrscheinlich damit zu tun, dass du nach dem Aufrufen vom Navigator mit dem du die CreateGameView zeigst nicht reloadest Fixed
flixcoo requested review from sneeex 2025-12-09 17:26:33 +00:00
flixcoo added 2 commits 2025-12-09 19:52:43 +00:00
Reverted method
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m2s
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
27ff599a88
sneeex requested changes 2025-12-09 20:18:10 +00:00
Dismissed
@@ -121,1 +130,3 @@
setState(() {});
setState(() {
_gameNameController;
});
Owner

bitte mit listener implementieren, außerdem hat das _gameNameController hier keinen Effekt, würde auch nur mit setState und leerem Inhalt gehen

bitte mit listener implementieren, außerdem hat das _gameNameController hier keinen Effekt, würde auch nur mit setState und leerem Inhalt gehen
flixcoo marked this conversation as resolved
flixcoo added 1 commit 2025-12-09 21:25:50 +00:00
Implemented listener
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m7s
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
0d0806dfbb
flixcoo requested review from sneeex 2025-12-09 21:25:56 +00:00
sneeex requested changes 2025-12-09 21:32:04 +00:00
Dismissed
sneeex left a comment
Owner

Player werden nach auswählen der Gruppe nicht sortiert

Player werden nach auswählen der Gruppe nicht sortiert
flixcoo requested review from sneeex 2025-12-09 22:59:38 +00:00
flixcoo added 1 commit 2025-12-09 22:59:39 +00:00
Fixed sorting problem
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Pull Request Pipeline / lint (pull_request) Successful in 2m4s
5d2fed74ac
sneeex requested changes 2025-12-10 08:23:50 +00:00
Dismissed
sneeex left a comment
Owner

Also bitte teste doch bevor du zur Review gibst einmal die Functionality und suche nach eventuellen Bugs, folgendes Problem:

//game erstellen klicken
//manuell spieler auswählen
//auf gruppe gehen
//gruppe auswählen
//spieler nicht mehr ausgewählt

Also bitte teste doch bevor du zur Review gibst einmal die Functionality und suche nach eventuellen Bugs, folgendes Problem: //game erstellen klicken //manuell spieler auswählen //auf gruppe gehen //gruppe auswählen //spieler nicht mehr ausgewählt
flixcoo added 1 commit 2025-12-10 11:38:48 +00:00
Fixed problem with player selection
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m7s
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
23cdddfbd9
flixcoo requested review from sneeex 2025-12-10 11:39:04 +00:00
sneeex requested changes 2025-12-10 12:27:30 +00:00
Dismissed
sneeex left a comment
Owner

chef also wirklich guckst du auch selber nach fehlern?
https://drive.google.com/file/d/119ZpheA7AE-STBo2C3anoNoWPQz8aBWi/view?usp=sharing

also es kann doch jetzt nicht sein dikker

chef also wirklich guckst du auch selber nach fehlern? https://drive.google.com/file/d/119ZpheA7AE-STBo2C3anoNoWPQz8aBWi/view?usp=sharing also es kann doch jetzt nicht sein dikker
flixcoo added 3 commits 2025-12-10 13:04:21 +00:00
Small refactoring
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
3b3d298ff5
flixcoo requested review from sneeex 2025-12-10 13:04:34 +00:00
sneeex requested changes 2025-12-10 20:10:46 +00:00
Dismissed
@@ -92,6 +113,7 @@ class _CreateGameViewState extends State<CreateGameView> {
Future.wait([_allGroupsFuture, _allPlayersFuture]).then((result) async {
groupsList = result[0] as List<Group>;
playerList = result[1] as List<Player>;
filteredPlayerList = List.from(playerList);
Owner

macht das nicht mehr sinn, rein logisch, die ausserhalb der future wait zu haben? weil die hat ja nichts in der future wait zu suchen oder?

macht das nicht mehr sinn, rein logisch, die ausserhalb der future wait zu haben? weil die hat ja nichts in der future wait zu suchen oder?
Author
Owner

theoreitisch ist das egal wo das ist hauptsache nach der future

theoreitisch ist das egal wo das ist hauptsache nach der future
Owner

ja ich weiß aber macht es nicht mehr sinn das ausserhalb zu machen

ja ich weiß aber macht es nicht mehr sinn das ausserhalb zu machen
Owner

so vom logischen

so vom logischen
flixcoo marked this conversation as resolved
Owner

und warum hast du den futurebuilder entfernt? weil die playerselection selber einen hat?

und warum hast du den futurebuilder entfernt? weil die playerselection selber einen hat?
Author
Owner

und warum hast du den futurebuilder entfernt? weil die playerselection selber einen hat?

Ja ich dachte erst dass das das problem behoben hat

> und warum hast du den futurebuilder entfernt? weil die playerselection selber einen hat? Ja ich dachte erst dass das das problem behoben hat
flixcoo added 1 commit 2025-12-10 20:41:43 +00:00
Moved statement
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
d0059b44a8
sneeex approved these changes 2025-12-10 20:42:29 +00:00
sneeex scheduled this pull request to auto merge when all checks succeed 2025-12-10 20:43:19 +00:00
sneeex merged commit 7b7c41b96c into development 2025-12-10 20:43:54 +00:00
sneeex deleted branch feature/3-creategameview-erstellen 2025-12-10 20:43:54 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Blocks
#78 Game zu Match umbenennen
liquid-development/game-tracker
Depends on
#20 GameHistoryView anpassen
liquid-development/game-tracker
Reference: liquid-development/game-tracker#67