Skip to content

Conversation

Zabuzard
Copy link
Member

@Zabuzard Zabuzard commented Nov 16, 2021

Overview

This marks the first release towards the Migrate existing commands objective, version 2.0.

It contains various commands, such as:

  • tag system
  • moderation commands
  • mod audit log
  • TeX command
  • voice chat activities

Plan

The plan is to release in a few days (Friday, 19. Nov. 2021), unless develop revealed any unstability in the recently merged features.

It is possible that the release gets delayed for a bit if any important features should be included short-term.

Checklist

Zabuzard and others added 30 commits October 8, 2021 11:55
Bumps JDA from 4.3.0_331 to 4.3.0_333.

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

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [jackson-dataformat-csv](https://github.com/FasterXML/jackson-dataformats-text) from 2.12.5 to 2.13.0.
- [Release notes](https://github.com/FasterXML/jackson-dataformats-text/releases)
- [Commits](FasterXML/jackson-dataformats-text@jackson-dataformats-text-2.12.5...jackson-dataformats-text-2.13.0)

---
updated-dependencies:
- dependency-name: com.fasterxml.jackson.dataformat:jackson-dataformat-csv
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps com.diffplug.spotless from 5.15.2 to 5.16.0.

---
updated-dependencies:
- dependency-name: com.diffplug.spotless
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
* Add Logviewer project
* Remove JPA and Hibernate, switch to Database-project
* Apply suggestions from code review
Bumps JDA from 4.3.0_333 to 4.3.0_334.

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

Signed-off-by: dependabot[bot] <[email protected]>
* Disable buttons after being used
* Added ephemeral explanation message to users clicking the buttons if they are not the user who triggered the command originally
Bumps a-vaadin-helper from 1.7.1 to 1.7.2.

---
updated-dependencies:
- dependency-name: org.vaadin.artur:a-vaadin-helper
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
*.db
*.db-shm
*.db-wal
The main method previously initialized the logger before setting the
system properties, leading to issues with the ForkJoinPool.
from: Available commands: [org.togetherjava.tjbot.commands.system.ReloadCommand@1ad777f, org.togetherjava.tjbot.commands.basic.PingCommand@5bbbdd4b, org.togetherjava.tjbot.commands.basic.DatabaseCommand@438bad7c]

to: Available commands: [reload, ping, db]
When using a DB path that has no direct parents, for example a relative path in the same directory like `local-database.db` (as recommended by the tutorial), the old code crashed since `getParent()` returned `null`.

This fixes it by:
1. using the absolute path before `getParent()`
2. checking the result of `getParent()` for `null`
- monthly
- only major updates (no patch, no minor)
- up to 10 at a time (default was 5)
… and org.togetherjava.logwatcher to org.togetherjava.tjbot.logwatcher to keep consistency
Co-authored-by: Zabuzard <[email protected]>
Co-authored-by: Heatmanofurioso <[email protected]>
* Add log severity highlighting to Logviewer #186
* introduced helper method `handleChecks` where all the preconditions are checked (reason, permissions, ...)
* this allows to give selected roles the ability to use moderation commands without giving them the ability to use them without the bot.
* for example staff assistants could kick users with the bot, but not without the bot
* that way, we can force them to go through the bot instead of using Discord GUI or other bots
* which gives us more control and visibility
@Zabuzard Zabuzard added the release Marks official releases and talks about them, i.e. merges from develop to master label Nov 16, 2021
@Zabuzard Zabuzard added this to the Migrate existing commands milestone Nov 16, 2021
@Zabuzard Zabuzard self-assigned this Nov 16, 2021
@Zabuzard Zabuzard requested a review from a team as a code owner November 16, 2021 11:06
@CLAassistant
Copy link

CLAassistant commented Nov 16, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
7 out of 10 committers have signed the CLA.

✅ Zabuzard
✅ krankkkk
✅ talentedasian
✅ illuminator3
✅ Tais993
✅ borgrel
✅ java-coding-prodigy
❌ RandomGHAcct
❌ I-Al-Istannen
❌ Yusuf Arfan Ismail


Yusuf Arfan Ismail seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@illuminator3
Copy link
Contributor

Could we please wait for the bytecode command? I'm pretty sure a lot of members would want that command on the main server.

@Zabuzard
Copy link
Member Author

Zabuzard commented Nov 16, 2021

Could we please wait for the bytecode command? I'm pretty sure a lot of members would want that command on the main server.

Have a rough ETA for that? That said, I also dont see any problem in having another release 1-2 weeks after.

@java-coding-prodigy
Copy link
Contributor

java-coding-prodigy commented Nov 17, 2021

Also, do wait for wolfram-alpha command, it is not very useful but it would suck if it got delayed by 6 months for this.
It will most likely get completed by next week anyway, so before bytecode command for sure.

@Tais993
Copy link
Member

Tais993 commented Nov 17, 2021

I also dont see any problem in having another release 1-2 weeks after.

No-one ever said 6 months and that's way too long.

We shouldn't constantly delay releases, so we'll see whenever that the Wolfram alpha command gets added.

Otherwise, 1/2 weeks

@java-coding-prodigy
Copy link
Contributor

We shouldn't constantly delay releases, so we'll see whenever that the Wolfram alpha command gets added.

Like I said, probably next week.

@Zabuzard
Copy link
Member Author

We shouldn't constantly delay releases, so we'll see whenever that the Wolfram alpha command gets added.

Like I said, probably next week.

Dont forget that features also have to be released and tested successfully for about a week on the develop server at least.

I doubt your feature will make it into this release. I also doubt free will make it. But thats okay, we will just ship another release in 2 weeks or so...

@java-coding-prodigy
Copy link
Contributor

So neither will bytecode make it I guess...

* void read -> readAndConsume
* T write -> writeAndProvide
@sonarqubecloud
Copy link

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 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Zabuzard Zabuzard merged commit 2087bfd into master Nov 19, 2021
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.