axelterizaki / erin-solstice-bot Goto Github PK

Thanks to typeORM who did that for us (see #2) we'll have to check that foreign key constraints are okay in some tables. I've seen some strange things like a ON DELETE NO ACTION for when you remove an item or user from the db in the inventory for example. A CASCADE should be here.

Useless thus needed : allowing users to list their skills, perhaps conditionning their number of skills by their level.

Skills can have different colors and such. It might be interesting to use the list of skills from the TWI wiki :

https://thewanderinginn.fandom.com/wiki/Skills#A

Since we're always getting/adding auto-assignable roles, the assignable boolean is useless, since it's always true.

As Discord is making a push for this, we need to update Discord.js to the latest version and completely rewrite the commands system.

When cloudflare or any fundamental stack of discord is down, bot will just rack up errors in log, and nothing'll happen.

By experience, it's better to try to reconnect once we receive an error; discord and cloudlare tends to be vbery SILENT when they're dead and very vocal to say "hey, you're offline, xoxo" when it's back.

A way for that is to store the heartbeat response time, and if that one goes too high, to reconnect, in client.on('error'

Note : we might not have that issue 'til a long time.

We're going to write SQL queries ourselves, they're pretty simple so it should be fine.

We have to rewrite what automatically creates SQLite databases upon joining a new server to make it work like in TypeoRM.

I haven't looked at Prisma event hoguh it seems to be the latest flavor in ORMs, I'm pretty much done with them now.