Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add send credential mails button for reviewers #209

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

himanshuvarandani
Copy link
Collaborator

PR description

Creating "Send Credential Mail" button for sending mails to reviewers.

Logic is still not correct as many calls are going to firebase function for sending mails, due to which some of them are failing.

@netlify
Copy link

netlify bot commented Mar 15, 2023

‼️ Deploy request for ssgsa-dev rejected.

Name Link
🔨 Latest commit 6ffa595

@pnijhara
Copy link
Collaborator

pnijhara commented Apr 1, 2023

Right now calls are async. This could be the reason some of them are failing. Try making calls synchronously and return when all of them are finished.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants