Fix webhook avatar, simplify message title

HangarMC · Jul 8, 2024 · 1a78c0b · 1a78c0b
1 parent cf4b92d
commit 1a78c0b
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/backend/src/main/java/io/papermc/hangar/components/webhook/model/DiscordWebhook.java b/backend/src/main/java/io/papermc/hangar/components/webhook/model/DiscordWebhook.java
@@ -1,13 +1,14 @@
 package io.papermc.hangar.components.webhook.model;
 
+import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonRawValue;
 import java.util.List;
 
 public record DiscordWebhook(
     String username,
-    String avatarUrl,
+    @JsonProperty("avatar_url") String avatarUrl,
     @JsonRawValue String embeds,
-    AllowedMentions allowedMentions
+    @JsonProperty("allowed_mentions") AllowedMentions allowedMentions
 ) {
 
     public DiscordWebhook(final String embeds) {

diff --git a/backend/src/main/java/io/papermc/hangar/components/webhook/service/WebhookService.java b/backend/src/main/java/io/papermc/hangar/components/webhook/service/WebhookService.java
@@ -31,7 +31,7 @@ public class WebhookService {
         [
           {
             "id": {{id}},
-            "title": "{{author}} / {{name}} is now available on Hangar!",
+            "title": "{{author}}/{{name}}",
             "description": "{{description}}",
             "color": 2326507,
             "fields": [],
@@ -50,7 +50,7 @@ public class WebhookService {
         [
           {
             "id": {{id}},
-            "title": "Version {{version}} for {{author}} / {{name}} has been released!",
+            "title": "Version {{version}} - {{author}}/{{name}}",
             "description": "{{description}}",
             "color": 2326507,
             "fields": [],

diff --git a/backend/src/main/java/io/papermc/hangar/service/internal/JobService.java b/backend/src/main/java/io/papermc/hangar/service/internal/JobService.java
@@ -61,7 +61,7 @@ public List<JobTable> getErroredJobs() {
     @Transactional
     public void schedule(final Job... jobs) {
         for (final Job job : jobs) {
-            logger.info("Scheduling job: {}", job);
+            this.logger.info("Scheduling job: {}", job);
             this.jobsDAO.save(job.toTable());
         }
     }

diff --git a/backend/src/main/java/io/papermc/hangar/service/internal/versions/VersionFactory.java b/backend/src/main/java/io/papermc/hangar/service/internal/versions/VersionFactory.java
@@ -472,7 +472,7 @@ private void verifyPendingPlatforms(final PendingVersion pendingVersion, final S
                 final Platform platform = entry.getKey();
                 final Set<String> versionsForPlatform = new HashSet<>(this.platformService.getFullVersionsForPlatform(platform));
                 if (!versionsForPlatform.containsAll(entry.getValue())) {
-                    throw new HangarApiException(HttpStatus.BAD_REQUEST, "version.new.error.invalidPlatformVersion", entry.getValue());
+                    throw new HangarApiException(HttpStatus.BAD_REQUEST, "version.new.error.invalidPlatformVersionList", entry.getValue());
                 }
             }
         }