Merge pull request #1 from vatsimnetwork/bug-fixes

Bug fixes

Author: vFlyingCowboy

cmd/main.go

@@ -1,27 +1,9 @@
 package main
 
 import (
-	"log"
-	"time"
-
 	"github.com/vatsimnetwork/go-ptd-bot/internal/bot"
-	"github.com/vatsimnetwork/go-ptd-bot/internal/config"
-
-	"github.com/getsentry/sentry-go"
 )
 
 func main() {
 	bot.Run()
-
-	if config.Env == "production" {
-		err := sentry.Init(sentry.ClientOptions{
-			Dsn:         config.SentryDSN,
-			Debug:       false,
-			Environment: config.Env,
-		})
-		if err != nil {
-			log.Fatalf("sentry.Init: %s", err)
-		}
-		defer sentry.Flush(2 * time.Second)
-	}
 }

internal/bot/client.go

@@ -4,6 +4,7 @@ import (
 	"log"
 	"os"
 	"os/signal"
+	"time"
 
 	"github.com/vatsimnetwork/go-ptd-bot/commands"
 	"github.com/vatsimnetwork/go-ptd-bot/internal/config"
@@ -24,6 +25,17 @@ func Session() (*discordgo.Session, error) {
 
 func Run() {
 	log.Print("Starting discord-bot-v2")
+	if config.Env == "production" {
+		err := sentry.Init(sentry.ClientOptions{
+			Dsn:         config.SentryDSN,
+			Debug:       false,
+			Environment: config.Env,
+		})
+		if err != nil {
+			log.Fatalf("sentry.Init: %s", err)
+		}
+		defer sentry.Flush(2 * time.Second)
+	}
 	s, err := Session()
 	if err != nil {
 		sentry.CaptureException(err)

internal/util/roles.go

@@ -1,6 +1,7 @@
 package util
 
 import (
+	"fmt"
 	"reflect"
 
 	"github.com/vatsimnetwork/go-ptd-bot/internal/api"
@@ -60,13 +61,15 @@ func ProcessMember(s *discordgo.Session, guildID string, m *discordgo.Member) {
 		return
 	}
 
-	roleIDs := make([]string, len(expectedRoles))
+	fmt.Println(actualRoles, expectedRoles)
+
+	roleIDs := new([]string)
 	for k := range expectedRoles {
-		roleIDs = append(roleIDs, k)
+		*roleIDs = append(*roleIDs, k)
 	}
 
 	_, err = s.GuildMemberEdit(guildID, m.User.ID, &discordgo.GuildMemberParams{
-		Roles: &roleIDs,
+		Roles: roleIDs,
 	})
 	if err != nil {
 		sentry.CaptureException(err)