feat: reverify command

gostmeaper commented

2026-05-26 04:37:59 +00:00

(Migrated from codeberg.org)

I wasn't able to test this thoroughly, but it ran cargo check and cargo build without issue. The new admitted_role type doesn't fail at least. I'm not sure how Anish was able to test this, but not my problem 😛 . BOOMPALA.

I wasn't able to test this thoroughly, but it ran `cargo check` and `cargo build` without issue. The new `admitted_role` type doesn't fail at least. I'm not sure how Anish was able to test this, but not my problem 😛 . BOOMPALA.

gostmeaper commented

2026-05-26 04:39:05 +00:00

(Migrated from codeberg.org)

#Iwantmyservertohavethisalready

gostmeaper commented

2026-05-26 05:31:25 +00:00

(Migrated from codeberg.org)

Idk it probably works or it'll break the bot 🤷

anish commented

2026-05-26 21:09:13 +00:00

(Migrated from codeberg.org)

@gostmeaper wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15890897:

#Iwantmyservertohavethisalready

can you explain what the point of the admitted role is? and what data its consuming

the scope of the bot is to just assign roles based on data from LDAP

@gostmeaper wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15890897: > #Iwantmyservertohavethisalready can you explain what the point of the admitted role is? and what data its consuming the scope of the bot is to just assign roles based on data from LDAP

gostmeaper commented

2026-05-26 21:12:42 +00:00

(Migrated from codeberg.org)

@anish wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15931286:

can you explain what the point of the admitted role is? and what data its consuming

the scope of the bot is to just assign roles based on data from LDAP

It's essentially another custom role you can assign along side first years. But I see what you mean. I suppose it would be sufficient to just swap everyone instead of adding the admitted role too. The only benefit would be we wouldn't need to go back and change the perms of admitted to first year.

@anish wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15931286: > can you explain what the point of the admitted role is? and what data its consuming > > the scope of the bot is to just assign roles based on data from LDAP It's essentially another custom role you can assign along side first years. But I see what you mean. I suppose it would be sufficient to just swap everyone instead of adding the admitted role too. The only benefit would be we wouldn't need to go back and change the perms of admitted to first year.

anish commented

2026-05-26 21:21:28 +00:00

(Migrated from codeberg.org)

@gostmeaper wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15931469:

@anish wrote in #1 (comment):

can you explain what the point of the admitted role is? and what data its consuming
the scope of the bot is to just assign roles based on data from LDAP

It's essentially another custom role you can assign along side first years. But I see what you mean. I suppose it would be sufficient to just swap everyone instead of adding the admitted role too. The only benefit would be we wouldn't need to go back and change the perms of admitted to first year.

wait i'm still confused on why the admitted role is necessary, what is the difference between that vs. the verified role

@gostmeaper wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15931469: > @anish wrote in #1 (comment): > > > can you explain what the point of the admitted role is? and what data its consuming > > the scope of the bot is to just assign roles based on data from LDAP > > It's essentially another custom role you can assign along side first years. But I see what you mean. I suppose it would be sufficient to just swap everyone instead of adding the admitted role too. The only benefit would be we wouldn't need to go back and change the perms of admitted to first year. wait i'm still confused on why the admitted role is necessary, what is the difference between that vs. the verified role

gostmeaper commented

2026-05-26 21:22:30 +00:00

(Migrated from codeberg.org)

it's not im getting rid of it 💀. We used it originally as a distinguisher between 29 and 30, but now it'll just be their class via LDAP.

gostmeaper commented

2026-05-26 21:29:42 +00:00

(Migrated from codeberg.org)

@anish Should be good now it's only the reverify command now. It'll just change people update people in waves.

gostmeaper commented

2026-05-26 22:03:14 +00:00

(Migrated from codeberg.org)

@anish fixed

gostmeaper commented

2026-05-26 22:10:49 +00:00

(Migrated from codeberg.org)

@anish it's ran them

anish commented

2026-05-26 22:47:20 +00:00

(Migrated from codeberg.org)

@gostmeaper can you rename the PR to something representative of the changes

gostmeaper commented

2026-05-26 23:07:22 +00:00

(Migrated from codeberg.org)

@anish anything else good sir

anish commented

2026-05-26 23:09:01 +00:00

(Migrated from codeberg.org)

@gostmeaper wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15935207:

@anish anything else good sir

nope looks good (unless it just breaks the bot)

@gostmeaper wrote in https://codeberg.org/ScottyLabs/discord-verify/pulls/1#issuecomment-15935207: > @anish anything else good sir nope looks good (unless it just breaks the bot)

Rows
Columns

feat: reverify command #1