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

Adding mail statistics in Badgeyay #1136

Closed
yashLadha opened this Issue Jun 29, 2018 · 1 comment

Comments

2 participants
@yashLadha
Member

yashLadha commented Jun 29, 2018

I'm submitting a ...

  • bug report
  • feature request

Current behaviour:

There is no mail statistics shown in the admin index page.

Expected behaviour:

There should be mail statistics shown in the mail index page

There are two ways to tackle it:

  • Using the firebase realtime database and as soon as a successful mail is sent it can add an entry with the date in it. This will also help to tackle the case for non successful emails sent, but in the admin we have to create the logic to parse the firebase realtime database objects to get the desired results and in this way we are more closely attaching to firebase as there is a limit on the usage of firebase realtime database (Storage limit).
  • Sending an API Request to our backend API and updating the entry in our database. This way we do not have to rely on the firebase services, but in the firebase free plan we cannot send an external request to an API except google own API's.

Views @djmgit @ParthS007 @gabru-md @abhi40308

@mariobehling

This comment has been minimized.

Member

mariobehling commented Jun 29, 2018

@yashLadha I am not sure what is the best way, but we can wait for others. It looks like both ways depend on Firebase. So, I tend to the "free plan" implementation.

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

@open-event-bot open-event-bot bot added the has-PR label Jul 1, 2018

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

yashLadha added a commit to yashLadha/badgeyay that referenced this issue Jul 1, 2018

ParthS007 added a commit that referenced this issue Jul 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment