-
Notifications
You must be signed in to change notification settings - Fork 189
[Media] Upload Digest Notifications (Override from CCNA) #8849
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
[Media] Upload Digest Notifications (Override from CCNA) #8849
Conversation
jeffersoncasimir
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as intended!
driusan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
String concatenation SQL needs to be fixed
|
Is there user documentation somewhere on how to set this up? RTD? Module README? It looks like it needs a cron job. |
…com/skarya22/Loris into 2023_07_19_Add_Media_Upload_Digests
| The media_upload_digest tool must be added to the Cron job for whatever frequency it should be run. | ||
| `php media_upload_digest.php [-days|-months|-years] [number] [-email (optional)]` | ||
|
|
||
| Further, each user that should receive the Media Upload Digest notifications should have the `media_upload_digest` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem to be listed in the permissions section above (around line 30)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated! Thanks
modules/media/README.md
Outdated
| `php media_upload_digest.php [-days|-months|-years] [number] [-email (optional)]` | ||
|
|
||
| Further, each user that should receive the Media Upload Digest notifications should have the `media_upload_digest` | ||
| permission, and `Media: Email Digest of Recently Uploaded Files` activated in the notifications tab. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is "the notifications tab"? I only see Upload/Edit and Browse
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Brief summary of changes
php media_upload_digest.php [-days|-months|-years] [number] [-email (optional)]in the tools directoryCCNA Overrides Involved:
Testing instructions
mysql -A < SQL/New_patches/2023_07_19_Add_Media_Digest.sqlExample of email