Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BOTM #16

Merged
merged 4 commits into from
Feb 9, 2025
Merged

BOTM #16

merged 4 commits into from
Feb 9, 2025

Conversation

realMorgon
Copy link
Contributor

BOTM Scoreboard.

  • Muss am Anfang einmal mit "/botm create" erstellt werden. Stellt sich danach bei Serverneustart selber wieder her, sobald der erste Spieler joint.
  • Bei der Erstellung müssen mind 4 Eintrage bestehen
    Database:
  • Ein table namens "botm" mit key "name" (varchar(255)) und "score"(int)
  • Ein table namens "botmlocation" mit key "axis" (varchar(255)) und "location" (varchar(255))

@realMorgon realMorgon self-assigned this Feb 8, 2025
@realMorgon realMorgon added the enhancement New feature or request label Feb 8, 2025
Copy link
Contributor

@Nudelsuppe42 Nudelsuppe42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alles gut, die einzigen zwei dinge:

  1. Wenn es geht für Nachrichten und Texte die Language API nutzen, damit wir es direkt Zweisprachig und in einer Datei haben
  2. Der Table für die Location ist glaube ich etwas zu komplex gelöst, entweder als einzelne row mit xyz oder noch einfacher einfach in der config.yml. Wir wollen so einzelne Table mit nur einem "config" eintrag wenn es geht verhindern

@Nudelsuppe42 Nudelsuppe42 removed the request for review from Le4nderS February 9, 2025 17:08
language API nutzen
removed debug
@realMorgon realMorgon merged commit ca4dfa4 into master Feb 9, 2025
1 check failed
@realMorgon realMorgon deleted the BOTM branch February 9, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants