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.

Sign up for GitHub

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

Jump to bottom

Continue/feature/reference gh #966

Closed
wants to merge 37 commits into from
Closed

Continue/feature/reference gh #966

wants to merge 37 commits into from

Conversation

ankitsmt211
Copy link
Member

@ankitsmt211 ankitsmt211 commented Nov 24, 2023 •
edited
Loading

continuing #599

Note: This merge will require Config to be changed on vps as well, otherwise bot crashes.

brief:

  • adds /github-search, which auto suggests a list of issues and PR to choose. It also accepts a title if you want to manually enter and select.
  • if a new message is of type #IssueNumberHere, it will reply back with an embed containing information about issue.
  • This is currently set to channel #general using config, can be tweaked to a particular channel.
  • More repositoriy Id's can be added to config to allow searching across multiple repositories.

Changes:

  • resolve merge conflicts
  • Java doc fixes
  • add avatar of user to embed
  • add date of creation to footer of embed
  • refactor embed builder for response

Embed on search/issue reference:

image

Tais993 and others added 21 commits September 30, 2022 23:48
@Tais993
Bugfix
490ec6f 
In BotCore:
Added onCommandAutoCompleteInteraction listener, this way autocompletion events will actually get forwarded.

Funny, didnt think of this during the previous PR, was probably too hasty.
@Tais993 @Taz03
Update application/src/main/java/org/togetherjava/tjbot/commands/syst…
aa58c07 
…em/BotCore.java

Co-authored-by: Tanish Azad <[email protected]>
@Taz03
fixed debug message
5f574d3
@illuminator3
add github referencing + github command
a9fef62
@illuminator3
make codeql and sonarcloud happy
9de6564
@illuminator3
spotless, *sigh*
1af1b54
@illuminator3
forgot these two
4325805
@illuminator3
remove mention when reference
d45db55
@illuminator3
aaaaaaaaaa
99ce678
@illuminator3
Merge branch 'develop' into feature/reference-gh
cad10d2
@illuminator3
fix compilation
a85abc2
@illuminator3
fix compilation x2
ef9989e
@illuminator3
apply spotless
1ba1418
@illuminator3
fix doc
3db6cfa
@illuminator3
requested changes
62b728d
@illuminator3
requested changes
3d2e48c
@illuminator3
requested changes
4c40202
@illuminator3 @Taz03
Update application/config.json.template
d98b829 
Co-authored-by: Tanish Azad <[email protected]>
@illuminator3 @Taz03
Update application/src/main/java/org/togetherjava/tjbot/commands/gith…
099f250 
…ub/GitHubCommand.java

Co-authored-by: Tanish Azad <[email protected]>
@illuminator3 @Taz03
Update application/src/main/java/org/togetherjava/tjbot/commands/gith…
b93dd7e 
…ub/GitHubReference.java

Co-authored-by: Tanish Azad <[email protected]>
@ankitsmt211
resolve conflicts
c7a8773
@ankitsmt211 ankitsmt211 requested review from a team as code owners November 24, 2023 16:13
@ankitsmt211 ankitsmt211 self-assigned this Nov 24, 2023
@ankitsmt211 ankitsmt211 added enhancement New feature or request new command Add a new command or group of commands to the bot priority: normal labels Nov 24, 2023
@Taz03 Taz03 changed the base branch from develop to feature/reference-gh November 25, 2023 03:25
@Taz03 Taz03 changed the base branch from feature/reference-gh to develop November 25, 2023 03:26
Zabuzard
Zabuzard requested changes Nov 25, 2023
View reviewed changes
database/build.gradle Outdated Show resolved Hide resolved
application/config.json.template Outdated Show resolved Hide resolved
application/config.json.template Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/commands/github/GitHubCommand.java Outdated Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/commands/github/GitHubCommand.java Outdated Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/commands/github/GitHubCommand.java Outdated Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/commands/github/GitHubReference.java Outdated Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/commands/github/GitHubReference.java Outdated Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/commands/github/GitHubReference.java Outdated Show resolved Hide resolved
application/src/main/java/org/togetherjava/tjbot/config/Config.java Outdated Show resolved Hide resolved
@ankitsmt211
making formatter a constant field instead of local var
0695af9
@ankitsmt211
update config template and verify allowed channels
109abf8 
* adds few channel patterns to be allowed in template
* refactors pattern matching for allowed channels
* helper method to match allowed channels
@ankitsmt211 ankitsmt211 requested a review from Zabuzard November 28, 2023 08:17
@ankitsmt211 ankitsmt211 closed this by deleting the head repository Dec 10, 2023
@ankitsmt211 ankitsmt211 mentioned this pull request Dec 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Zabuzard Zabuzard Awaiting requested review from Zabuzard

Assignees

@ankitsmt211 ankitsmt211

Labels
enhancement New feature or request new command Add a new command or group of commands to the bot priority: normal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ankitsmt211 @Zabuzard @Tais993 @Taz03 @illuminator3