-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Updated Discord Bot Code #393
Open
Frank-Lom
wants to merge
16
commits into
NelsonDane:main
Choose a base branch
from
Frank-Lom:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+189
−153
Commits on Oct 2, 2024
-
Removed any references I could find restricting the discord bot to a specific channel. I swapped it to utilize slash (/) commands and operate under a user installed bot rather than a guild installed bot. Discord has this as an official option when creating a bot and I felt having to have an entire server dedicated for a single bot was unnecessary so now it can be used anywhere at all. All messages are sent with ephemeral set to true if not in DMs with the bot so the bot commands can be executed in literally any server or in a DM with the bot itself and nobody else can see it. If commands are executed in a server, responses for holdings and transactions are DMed to you by the bot instead of sent in the server, even though ephemeral would be set to true so they would be hidden I felt it'd be better to have it in DMs either way. Holdings command is tested and working, Transactions command should function just as normal but have not fully tested since the market is closed. I did try it and it returned errors properly about what ticker I chose, the quantity, and the broker, but it would be worth testing again while the market is open before fully releasing. Dry is set to a default of True for transaction commands unless overrode during executing the command.
Configuration menu - View commit details
-
Copy full SHA for 51fbf44 - Browse repository at this point
Copy the full SHA 51fbf44View commit details -
Configuration menu - View commit details
-
Copy full SHA for f8a3bc6 - Browse repository at this point
Copy the full SHA f8a3bc6View commit details -
guides, example env, and dm command
Updated guides and example env file Added a /dm command to start a dm with the bot if needed Removed unused DISCORD_CHANNEL variable from example env Added dry default state to env file so it can be defaulted to whatever the user wants (DISCORD BOT ONLY)
Configuration menu - View commit details
-
Copy full SHA for 9de26d4 - Browse repository at this point
Copy the full SHA 9de26d4View commit details -
Cleaned up minor code suggestions
Trimmed leftover whitespace Removed unused imports Adjusted line spacing
Configuration menu - View commit details
-
Copy full SHA for b5edb3f - Browse repository at this point
Copy the full SHA b5edb3fView commit details -
Removed default dry, merged ping and dm commands
Removed default dry toggle Ping command now dms user so no need for /dm
Configuration menu - View commit details
-
Copy full SHA for 9e7386c - Browse repository at this point
Copy the full SHA 9e7386cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 02e5176 - Browse repository at this point
Copy the full SHA 02e5176View commit details -
Configuration menu - View commit details
-
Copy full SHA for 145e4d9 - Browse repository at this point
Copy the full SHA 145e4d9View commit details
Commits on Oct 3, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 5d0febd - Browse repository at this point
Copy the full SHA 5d0febdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4af5d90 - Browse repository at this point
Copy the full SHA 4af5d90View commit details
Commits on Oct 4, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 4df1164 - Browse repository at this point
Copy the full SHA 4df1164View commit details -
Configuration menu - View commit details
-
Copy full SHA for 91be3a1 - Browse repository at this point
Copy the full SHA 91be3a1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8e8feee - Browse repository at this point
Copy the full SHA 8e8feeeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 906d355 - Browse repository at this point
Copy the full SHA 906d355View commit details -
Configuration menu - View commit details
-
Copy full SHA for 91e22f1 - Browse repository at this point
Copy the full SHA 91e22f1View commit details -
Configuration menu - View commit details
-
Copy full SHA for df8dfb3 - Browse repository at this point
Copy the full SHA df8dfb3View commit details
Commits on Oct 5, 2024
-
Configuration menu - View commit details
-
Copy full SHA for a4015cf - Browse repository at this point
Copy the full SHA a4015cfView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.