Added Database Backup Photos to Readme file#177
Conversation
admin_pages_dev: Added Database Backup Photos To Readme file
Reviewer's GuideIntroduces visuals for the database backup workflow by adding CLI, cronjob, script code, and S3 storage screenshots to the README. Flow Diagram of Documented Database Backup Workflowgraph TD
subgraph "Backup Triggers"
direction LR
CLI["User via CLI"]
Cronjob["Scheduled Cronjob"]
end
Script["Backup Script Execution"]
Database["Database (Source)"]
S3["S3 Bucket (Destination)"]
CLI --> Script;
Cronjob --> Script;
Script -- "1. Reads data from" --> Database;
Script -- "2. Writes backup to" --> S3;
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey @Riyad-Murad - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 2 issues found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
|
||
| | Database Backup CLI Script Run | Database Backup Cronjob | | ||
| | --------------------------------------- | ------------------------------------- | | ||
| |  |  | |
There was a problem hiding this comment.
suggestion: Enhance image alt text for improved accessibility and clarity.
The alt text for these images is too brief. Use more descriptive text, e.g., “Screenshot of Database Backup CLI Script execution” and “Demonstration of Database Backup Cronjob setup,” to improve accessibility.
| |  |  | | |
| |  |  | |
|
|
||
| | Database Backup Script Code | Database Backup S3 | | ||
| | --------------------------------------- | ------------------------------------- | | ||
| |  |  | |
There was a problem hiding this comment.
suggestion: Enhance image alt text and align ![DBBackupS3] with its header.
Use descriptive alt texts like “Code snippet for the database backup script” and “View of S3 bucket with database backups,” and adjust the DBBackupS3 alt to exactly match its header for clarity.
| |  |  | | |
| |  |  | |
Summary by Sourcery
Add visual documentation for the database backup process to the README
Documentation: