sam/Catalogger.NET
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: don't specify duplicate key_roles in guild repository, let PK API service read strings as numbers

Browse source
This commit is contained in:
sam 2024-10-28 14:57:05 +01:00
parent ff92c5f335
commit 56af787e57
Signed by: sam
GPG key ID: 5F3C3C1B3166639D
3 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Catalogger.Backend/Database/DatabaseMigrator.cs
View file

@ -65,9 +65,11 @@ public class DatabaseMigrator(ILogger logger, IClock clock, DatabaseConnection c
await tx.CommitAsync();
}
private async Task ExecuteMigration(DbTransaction tx, string migrationName)
private async Task ExecuteMigration(DbTransaction tx, string migrationName, bool up = true)
{
var query = await GetResource($"{RootPath}.Migrations.{migrationName}.up.sql");
var query = await GetResource(
$"{RootPath}.Migrations.{migrationName}.{(up ? "up" : "down")}.sql"
);
// Run the migration
await conn.ExecuteAsync(query, transaction: tx);

2
Catalogger.Backend/Database/Repositories/GuildRepository.cs
View file

@ -51,7 +51,7 @@ public class GuildRepository(ILogger logger, DatabaseConnection conn)
public async Task AddGuildAsync(ulong id) =>
await conn.ExecuteAsync(
"""
insert into guilds (id, key_roles, banned_systems, key_roles, channels)
insert into guilds (id, key_roles, banned_systems, channels)
values (@Id, array[]::bigint[], array[]::text[], array[]::bigint[], @Channels)
on conflict do nothing
""",