Skip to content

Conversation

ez314
Copy link
Member

@ez314 ez314 commented Jun 1, 2022

Adding modals to the bot, wherever useful.

Aim to fully resolve #71

@ez314 ez314 added the wip Work in progress, do not merge yet label Jun 1, 2022
@ez314 ez314 force-pushed the feature/modals branch from dab7d9f to 4f2610e Compare June 4, 2022 03:36
@@ -22,12 +22,12 @@
"homepage": "https://github.com/jafrilli/acm-bot#readme",
"dependencies": {
"@discordjs/builders": "^0.12.0",
"@discordjs/rest": "^0.1.0-canary.0",
Copy link
Member Author

Choose a reason for hiding this comment

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

Was using an alpha version of this earlier, but stable version works now

});
for (const manager of Object.values(this.managers)) {
await manager.init();
}
Copy link
Member Author

Choose a reason for hiding this comment

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

Some managers depend on others to start up first (error handling, databases, etc. should be first)

Copy link
Member Author

Choose a reason for hiding this comment

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

should probably leave a comment here

);
await interaction.reply({
content: "Command execution failed. Please contact a bot maintainer...",
ephemeral: true
Copy link
Member Author

Choose a reason for hiding this comment

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

Made this ephemeral to hide failed executions from public, in case someones trying to use an anon slash command or smth.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wip Work in progress, do not merge yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement modal functionalities
1 participant