search

marcelropos / HM-DiscordBot

A Discordbot by and for first semester students of HM.
8 stars 5 forks source link

add id use for verify command #123

Closed naxey closed 1 year ago

naxey commented 1 year ago

:D

maxwai commented 1 year ago

Please some explanation why this change is needed. What use case this solves.

naxey commented 1 year ago

It adds the option to use the userID instead of a mention for verifying. Its intent is to provide quality of life. I've already talked with @marcelropos about this change

maxwai commented 1 year ago

It adds the option to use the userID instead of a mention for verifying. Its intent is to provide quality of life. I've already talked with @marcelropos about this change

Ok, then it is fine. Just try to include such information in the initial Pull Request Comment in the future.

Fix the remaining things and we are good.

I also changed the base Branch to develop as we also need to test it before deploying (it couldn't be done before as the develop branch was behind master sorry)

marcelropos commented 1 year ago

fix that on develop