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

Release v.2.4 #360

Merged
merged 27 commits into from
Feb 3, 2022
Merged

Release v.2.4 #360

merged 27 commits into from
Feb 3, 2022

Conversation

Zabuzard
Copy link
Member

@Zabuzard Zabuzard commented Feb 3, 2022

Overview

This release is another step towards the command migration. Also, it polishes a lot of the existing commands.

Highlights in this release are:

  • adding /top-helpers
  • adding reply-to option to /tag

Changelist

Commands

  • adding /top-helpers
  • adding reply-to option to /tag
  • UX for audit logs (removed title)
  • UX for free/busy messages (hint how to use /free)
  • bugfix for /vc-activity when giving high values
  • bugfix for /tag-manage raw (leading spaces are not shown)

Platform

  • bugfix with broken encoding in unit tests
  • adding message and event receivers
  • adding routines
  • adding config template

Budbomber and others added 27 commits January 18, 2022 13:12
* bugfix causing crash when using high max-age on /vc-activity
* UX improvements on max-age, users now enter days instead of seconds
Bumps JDA from 4.4.0_351 to 4.4.0_352.

---
updated-dependencies:
- dependency-name: net.dv8tion:JDA
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
instead of escaped markdown in an embed which leads to bugs
- Renamed Commands to Features
- Renamed CommandSystem to BotCore
- Integrated new MessageReceiver and EventReceiver capabilities
- Migrated RejoinMuteListener to new EventReceiver
nobody except /free needed it. /free now uses `EventReceiver` which also propagates `ReadyEvent` already.
* Empty lines for readability
* Slightly rephrased a user facing string
* changed the layout of the methods
* fixed style issues
* fixed various design issues
* streamlined the code more

the logic itself merely changed
* with proper java.time computation instead of just "last 30 days"
@Zabuzard Zabuzard added the release Marks official releases and talks about them, i.e. merges from develop to master label Feb 3, 2022
@Zabuzard Zabuzard added this to the Migrate existing commands milestone Feb 3, 2022
@Zabuzard Zabuzard self-assigned this Feb 3, 2022
@Zabuzard Zabuzard requested review from a team as code owners February 3, 2022 08:14
Copy link
Member

@Tais993 Tais993 left a comment

Choose a reason for hiding this comment

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

+1

@Zabuzard Zabuzard merged commit ac1eb59 into master Feb 3, 2022
@sonarcloud
Copy link

sonarcloud bot commented Feb 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Marks official releases and talks about them, i.e. merges from develop to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants