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

Implement Depositor Notifications #1020

Open
1 task
Tracked by #1005 ...
ShanaLMoore opened this issue Apr 26, 2024 · 16 comments
Open
1 task
Tracked by #1005 ...

Implement Depositor Notifications #1020

ShanaLMoore opened this issue Apr 26, 2024 · 16 comments

Comments

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Apr 26, 2024

Summary

BLOCKED until we have a conversation with the client.

SOW

The goal is to implement email notifications for some activities/trigers in the Hyku application. That include:

  • Notifications to authors related to views and downloads of their works.

Acceptance Criteria

  • Email notifications to author related to views and downloads of their works are set up
    • (Jeremy here: the notifications of this bullet point are not handled in the "Implementation Way Finding" and require a different approach.)

Screenshots or Video

Testing Instructions

Notes

Spike results: #1005 (comment)
Implementation brainstorm: #1005 (comment)

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented May 6, 2024

Notes from Meeting with PITT/PALS

Topic: Notifications for Downloads and Views

Should we email Authors or depositors?

  • Next phase: Email the actual author
    • Expand contributor to represent an actual author. Ref British library's forms
  • For now, we’ll email depositors.

Acceptance Criteria:

  • Depositor should get a monthly email.
  • Email aggregates the counts of downloads and views of the work(s) they've deposited.

How often?

  • Ideally, Feature Parity with digital commons.
    • Sends monthly round up of views and downloads to the depositor. (Or author)

Sample:

Dear Author,
You had 1 new download in December 2023 across your 2 papers in MUShare. Your current readership:

26 Total Downloads
Visit My Dashboard Icon VISIT MY
DASHBOARD

These monthly reports are provided to you by bepress on behalf of MUShare. For questions, comments, or to add more content and increase your readership and visibility as an author, please contact your repository administrator:

Why do you want this feature?

  • Makes them feel good about engaging with the platform
  • Give them the "I did something and people noticed" dopamine hit
  • Don’t need a call to action
  • Don't worry about increasing engagement metric.

@jillpe
Copy link

jillpe commented May 16, 2024

blocked by GA4, valkyrie work and email notifications rework

@ShanaLMoore
Copy link
Contributor Author

Google Analytics PR

@jillpe jillpe changed the title Implement Author Notifications Implement Depositor Notifications May 28, 2024
@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented May 31, 2024

@aahurford or @ndroark Should an email still be sent even if the depositor has received no new views or file downloads?

We are developing based off the provided sample:

Sample:

Dear Author,
You had 1 new download in December 2023 across your 2 papers in MUShare. Your current readership:

26 Total Downloads
Visit My Dashboard Icon VISIT MY
DASHBOARD

These monthly reports are provided to you by bepress on behalf of MUShare. For questions, comments, or to add more content and increase your readership and visibility as an author, please contact your repository administrator:

@ShanaLMoore ShanaLMoore moved this to In Development in palni-palci May 31, 2024
@ShanaLMoore ShanaLMoore self-assigned this May 31, 2024
@aahurford
Copy link
Collaborator

aahurford commented May 31, 2024 via email

@jillpe
Copy link

jillpe commented Jun 13, 2024

@ShanaLMoore
To Do:
test if pals ga4 set up and working in staging.
For QA we will send a manual email to confirm it displays how we want and then will need to wait for the monthly email to confirm automation works

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Jun 13, 2024

Blocked - find tenant with GA4 tracking in staging

TODO: Shana to Pair with Rob for setup instructions

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Jun 27, 2024

Spike ticket is needed. What is supposed to read and increment the user counts?

rob's comment: the questions really are: are the events being captured. if yes, are the events being read from the API. if yes, are the events being cached correctly in the reports for display and use

@jillpe jillpe moved this from Code Review to SoftServ QA in palni-palci Jul 1, 2024
@jillpe jillpe removed the Blocked label Aug 26, 2024
@ShanaLMoore
Copy link
Contributor Author

QA RESULTS: ⚠️ PENDING

This appears to be working. Tested with THIS TENANT,

The email matches the details of the dashboard.

However, I am clicking around a bunch in another tenant. (demo tenant) It'll take about a day for the events to get tracked with GA4. Those stats and email should be different. right now it's kind of odd that the stats are the same.

DEMO profile

Image

EMAIL

Image

ADMIN DASHBOARD

Image

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Sep 3, 2024

The counts don't appear to be changing.

At the turn of the month, emails went out regardless if the count was zero. (they should only email when counts are > 0)

Additionally, emails had no tenants specified in some of the emails.

Image

UPDATE: the tenant must have an application name filled in for the mail to reference

Image

if it's empty, maybe reference the url/cname

@ShanaLMoore ShanaLMoore moved this from SoftServ QA to Ready for Development in palni-palci Sep 3, 2024
@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Sep 3, 2024

TODOS:

  • - Hyrax: content-id => content_id to track file download stats (Shana and LaRita) + submodule update
  • - add fall backs for application name (Shana and LaRita)
  • - don't send email if 0 (Shana and LaRita)
  • - all download buttons needs 'file_download' as id for tracking (Shana and LaRita)
    • missing from:

Image

  • update knapsack submodule (LaRita)

  • - tenantmaintenancejob issue (Rob)

    • re scheduling jobs issue - UserStatCollectionJob

@ShanaLMoore ShanaLMoore assigned orangewolf and unassigned kirkkwang Sep 3, 2024
@ShanaLMoore ShanaLMoore removed this from palni-palci Sep 3, 2024
ShanaLMoore added a commit to samvera/hyrax that referenced this issue Sep 3, 2024
In Google Analytics we were missing the tracking for download events. This is because there was a typo in content_id.

Issue:
- scientist-softserv/palni-palci#1020
ShanaLMoore added a commit to samvera/hyku that referenced this issue Sep 3, 2024
ID must be changed so to be consistent with google analytics tracking.

Issue:
- scientist-softserv/palni-palci#1020
@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Sep 5, 2024

QA RESULTS: ❌ NEEDS REWORK

tested on STAGING

Starting with the following stats. I will then click around and download a bunch of files today. I expect to see updated stats tomorrow. It takes 24 hrs for the tracking to update.

Image

TESTED 4 DAYS LATER

Counts are the same

Image

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Sep 16, 2024

Today I noticed a change in the profile count, which indicates ga4 is working afterall:

Image

In staging I ran DepositorEmailNotificationJob.perform_now

Image

Next Day Results after clicking around/downloading things

Image

side note

this was seen in adventist: Image

@ShanaLMoore ShanaLMoore removed their assignment Sep 17, 2024
@ShanaLMoore
Copy link
Contributor Author

@orangewolf Could you please QA this ticket and make sure things are working as you expect? I'm not convinced this is working right.

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Sep 23, 2024

I checked maildev today and was surprised to see 129+ new emails that were sent on sep 20th. (it's not a new month, why were they sent?)

Also, even though each tenant is different, they all say the same thing.

Image

@ShanaLMoore
Copy link
Contributor Author

ShanaLMoore commented Oct 8, 2024

A new batch of monthly emails were sent on 9-30-24 and the contents all look different now. None of the ones spot checked had a count for File Download.

Image

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

No branches or pull requests

6 participants