platform icon indicating copy to clipboard operation
platform copied to clipboard

Add resend functionality for verification emails in AuthzController

Open harshitghagre opened this issue 7 months ago • 10 comments

harshitghagre avatar Apr 22 '25 08:04 harshitghagre

@GHkrishna check this PR

harshitghagre avatar Apr 22 '25 09:04 harshitghagre

Can you please have a look at the flow from controller, since the verification changes that you are doing are for verification of credential which is not authentication flow

okay I'll check it

harshitghagre avatar Apr 22 '25 10:04 harshitghagre

BTW loved your enthusiasm and willingness to contribute, @harshitghagre

GHkrishna avatar Apr 23 '25 10:04 GHkrishna

BTW loved your enthusiasm and willingness to contribute, @harshitghagre

Thanks @GHkrishna . By profession, I'm not a developer, but I'm trying to fix the issue because I'm getting interested in how the resend mail functionality works,I'm still learning and working on resolving the issue.

harshitghagre avatar Apr 24 '25 06:04 harshitghagre

By profession, I'm not a developer, but I'm trying to fix the issue because I'm getting interested in how the resend mail functionality works,I'm still learning and working on resolving the issue.

That's even better, we need more people like you. You're going good. All the best.

GHkrishna avatar Apr 24 '25 06:04 GHkrishna

By profession, I'm not a developer, but I'm trying to fix the issue because I'm getting interested in how the resend mail functionality works,I'm still learning and working on resolving the issue.

That's even better, we need more people like you. You're going good. All the best.

could I know which SMTP server are you using ? is it self host or third party?

harshitghagre avatar Apr 24 '25 06:04 harshitghagre

For emails, we don't have any self-hosted servers. Instead we use Sendgrid, a SaaS.

You can refer the docs here: https://docs.credebl.id/docs/getting-started/local-deployment/platform#sendgrid

GHkrishna avatar Apr 24 '25 07:04 GHkrishna

@GHkrishna I made some changes please check it

harshitghagre avatar Apr 25 '25 05:04 harshitghagre

@GHkrishna some of the previous commits is mergred in currrent commit of authz.controller.ts files so it will be better that i'll create new PR in which I do the changes

harshitghagre avatar Apr 25 '25 09:04 harshitghagre

it will be better that i'll create new PR in which I do the changes Sounds right to me, closing the PR

GHkrishna avatar Jun 06 '25 06:06 GHkrishna