MidgardVTT-Entwicklung/foundry-vtt-system-midgard5
7
0
Fork 0
Code Issues 16 Pull Requests Actions Projects 1 Releases 8 Wiki Activity

#18 Remove ContainerId from containers #29

Merged
Byroks merged 1 commits from #18-remove-unnecessary-containerId into develop 2024-01-15 14:15:32 +01:00
Conversation 1 Commits 1 Files Changed 1
+20 -23
Byroks commented 2024-01-08 12:11:45 +01:00
Owner
Copy Link
Copy Source

Changes

  • remove containerId variable from containers as it's not needed
Changes + remove containerId variable from containers as it's not needed
Byroks added 1 commit 2024-01-08 12:11:46 +01:00
#18 Remove ContainerId from containers fbbd55c420 
Changes
 + remove containerId variable from containers as it's not needed
Byroks requested review from Le-Frique 2024-01-08 12:11:52 +01:00
Byroks requested review from oskaloq 2024-01-08 12:11:52 +01:00
Byroks requested review from ruebe5w 2024-01-08 12:11:52 +01:00
oskaloq approved these changes 2024-01-15 13:46:00 +01:00
oskaloq left a comment
Member
Copy Link
Copy Source

Jetzt verstehe ich, was Du gemeint hast: Die containerId bei den Containern war dafür gedacht, dass man auch Container selbst in andere Container packen kann (zB Truhe in Haus oder Geldbeutel in Rucksack). Ist aber wohl besser, wenn wir das erst einmal weglassen. Macht die Berechnung von Wert und Lasten vermutlich einfacher ...

Jetzt verstehe ich, was Du gemeint hast: Die containerId bei den Containern war dafür gedacht, dass man auch Container selbst in andere Container packen kann (zB Truhe in Haus oder Geldbeutel in Rucksack). Ist aber wohl besser, wenn wir das erst einmal weglassen. Macht die Berechnung von Wert und Lasten vermutlich einfacher ...
Byroks merged commit b5b75aa2c6 into develop 2024-01-15 14:15:32 +01:00
Byroks deleted branch #18-remove-unnecessary-containerId 2024-01-15 14:15:32 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
ruebe5w
Le-Frique
oskaloq
Labels
Clear labels
bug
Something is not working
 feature
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 regelfehler
Funktionalität stimmt nicht mit Regeln überein
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Byroks Le-Frique Masek0815 korstefan14 oskaloq ruebe5w veilza
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MidgardVTT-Entwicklung/foundry-vtt-system-midgard5#29