Skip to content

Conversation

Tais993
Copy link
Member

@Tais993 Tais993 commented Jan 20, 2022

Pull-request

Changes

  • Existing code
  • New feature

Closes Issue: #311

Description

Currently a draft, issues mentioned in the old PR #329 will be resolved.

To-do:

  • Rebase
  • Apply spotless
  • ....

talentedasian and others added 15 commits December 15, 2021 23:38
  * Add a "guildId" config in Config.json for later use in finding the
    moderators of the guild.
  * Initially, the bot response should've been ephemeral but since I was
    using a selection menu, the way I was disabling it does not work if
    I set the bot response to ephemeral.
and use it for subsequent invocation of the command.
highly unlikely, send an ephemeral message and log the unusual behavior
…. This also fixes the failed interaction by discord
  the /modmail command.

* Introduce a utility class for use by other classes when interacting with the ModMailCommand.

* Some javadoc.
  Replace all loadOptions with the onReady event but still delegating to
  the ModmailUtils to query discord.

* Move ReloadModmailCommand from a separate class to the ModmailCommand
  class.
  * Add a "guildId" config in Config.json for later use in finding the
    moderators of the guild.
  * Initially, the bot response should've been ephemeral but since I was
    using a selection menu, the way I was disabling it does not work if
    I set the bot response to ephemeral.
and use it for subsequent invocation of the command.
highly unlikely, send an ephemeral message and log the unusual behavior
…. This also fixes the failed interaction by discord
  the /modmail command.

* Introduce a utility class for use by other classes when interacting with the ModMailCommand.

* Some javadoc.
  Replace all loadOptions with the onReady event but still delegating to
  the ModmailUtils to query discord.

* Move ReloadModmailCommand from a separate class to the ModmailCommand
  class.
…ture/modmail

# Conflicts:
#	application/src/main/java/org/togetherjava/tjbot/commands/Commands.java
@Tais993 Tais993 added enhance command Modify or improve an existing command or group of commands of the bot priority: normal labels Jan 20, 2022
@Tais993 Tais993 added this to the Improvement phase 1 milestone Jan 20, 2022
@Tais993 Tais993 self-assigned this Jan 20, 2022
…o feature/modmail

� Conflicts:
�	application/src/main/java/org/togetherjava/tjbot/commands/Commands.java
@Zabuzard Zabuzard linked an issue Feb 3, 2022 that may be closed by this pull request
@github-actions
Copy link

This pull request is stale because it has been open 30 days with no activity. Remove stale label, comment or add the valid label or this will be closed in 5 days.

@github-actions github-actions bot added stale inactivity-closed Issues that have been closed due to inactivity, but are otherwise valid and might be reopened later labels Mar 11, 2022
@github-actions github-actions bot closed this Mar 16, 2022
@Tais993 Tais993 deleted the feature/modmail branch October 12, 2022 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhance command Modify or improve an existing command or group of commands of the bot inactivity-closed Issues that have been closed due to inactivity, but are otherwise valid and might be reopened later priority: normal stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create /modmail command

2 participants