Skip to content

Add Geschenkpapier as loot item#605

Merged
twobiers merged 15 commits into
masterfrom
geschenkpapier
May 12, 2026
Merged

Add Geschenkpapier as loot item#605
twobiers merged 15 commits into
masterfrom
geschenkpapier

Conversation

@twobiers
Copy link
Copy Markdown
Collaborator

Mit Geschenkpapier lässt sich ein zufälliges Item aus dem Inventar in ein Geschenk verwandeln.

Ich hab leider keine schöne Möglichkeit gefunden ein Item auszuwählen, weil es immer ein Limit von 25 optionen max. gibt. Vielleicht kann man das noch irgendwie in das /gegenstand benutzen dynamisch einbauen. Mit Modals hab ich nichts gefunden.

Alternative zum direkten droppen wäre, dass man ein eingepacktes Geschenk ins Inventar kriegt, fand ich aber unnötig komplex. Wenn man ein Geschenk einpackt, will man es vermutlich auch zeitnah droppen.

@twobiers twobiers requested a review from holzmaster May 11, 2026 15:04
Comment thread src/service/lootData.ts Outdated
@github-actions
Copy link
Copy Markdown
Contributor

Hallo, hier kannst du jetzt deine tolle neue Funktion testen: https://discord.gg/pJjfF6n2ec

Comment thread src/service/lootData.ts Outdated
Comment thread src/service/lootData.ts Outdated
Comment thread src/service/lootDrop.ts Outdated
@twobiers
Copy link
Copy Markdown
Collaborator Author

Ich hab in den PR noch ein Debug Command eingeschleust, damit ich den PR testen kann. Kann ich auch in einen separaten PR cherry-picken später, wenn nötig

Comment thread src/commands/devcommands/loot-drop.ts Outdated
Comment thread src/service/command.ts Outdated
Comment thread src/service/lootDrop.ts Outdated
@holzmaster
Copy link
Copy Markdown
Collaborator

Hab das wrapable-attribut schonmal in den master gezogen. Das wird denk ich unverändert bleiben

@holzmaster
Copy link
Copy Markdown
Collaborator

Hab die dev-commands schonmal in den Master gezogen

Comment thread src/service/lootData.ts
Comment thread src/service/lootData.ts Outdated
Comment thread src/service/lootData.ts Outdated
Comment thread src/service/lootDrop.ts
Comment thread src/service/lootDrop.ts Outdated
Comment thread src/service/lootDrop.ts Outdated
Comment thread src/service/lootDrop.ts
@holzmaster
Copy link
Copy Markdown
Collaborator

Wirkt etwas leichter als das mit der strategy. Mir gefällt, dass der Randomize-Part jetzt wo anders ist

@twobiers twobiers merged commit 89449ed into master May 12, 2026
6 checks passed
Comment thread src/service/lootData.ts
await interaction.reply({
content: "Du hast nichts, was du einpacken könntest! 😢",
});
return false;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@twobiers soll das geschenkpapier aus dem Inventar gelöcsht werden, wenn die benutzung nicht funktioniert hat? Fänd ich komisch

@holzmaster holzmaster deleted the geschenkpapier branch May 15, 2026 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants