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

feat: add set currency command #2925

Merged
merged 5 commits into from
Dec 25, 2024

Conversation

CKY-
Copy link
Collaborator

@CKY- CKY- commented Dec 12, 2024

Description of the Change

adds the set currency command

{4EA7275B-BC97-4BF2-9E3C-3F17DA236C3B}
{044BC2E8-357B-423A-8898-9E36D1611980}

Applicable Issues

#2874

Testing

Screenshots

@CKY- CKY- requested a review from ebiggz as a code owner December 12, 2024 23:39
Copy link
Member

@CaveMobster CaveMobster left a comment

Choose a reason for hiding this comment

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

CKY, this is looking great! I'm a major pain in the butt when it comes to capitalization and such though, so two very small things but other than that it's awesome. 😄

src/backend/currency/currency-command-manager.ts Outdated Show resolved Hide resolved
src/backend/currency/currency-command-manager.ts Outdated Show resolved Hide resolved
@CaveMobster CaveMobster merged commit d24d9c9 into crowbartools:v5 Dec 25, 2024
1 check passed
Alastor-git pushed a commit to Alastor-git/Firebot that referenced this pull request Dec 27, 2024
* feat: add set currency command

* fix capitalization

---------

Co-authored-by: CaveMobster <[email protected]>
@CKY- CKY- deleted the v5-set-user-currency-command branch January 1, 2025 18:59
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