-
Notifications
You must be signed in to change notification settings - Fork 6
image refresh and ministry articles re-steuctire #421
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
WalkthroughThe pull request modifies the documentation for Field Service Reports, focusing on enhancing clarity regarding report statuses and management. Key updates include the replacement of the "Received" status with "Pending verification," adjustments to the "Not Submitted" status for consistency, and clarifications on managing late reports. The document specifies that submitted reports cannot be edited, while late reports remain editable, and outlines the process for marking reports as late. The overall structure and supporting images remain unchanged. Changes
Possibly related PRs
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
organized/docs/how-to-use/reports/field-service-reports.md (2)
Line range hint
66-73: Enhance readability of late reports section with bullet points.Consider restructuring the late reports explanation for better readability.
Organized makes it easy for secretary to manage late reports. If a report is submitted after the deadline for the branch office, you can mark it as 'late'. Here's how it works: -- **Marking as late report:** If a report comes in after the deadline, simply mark it as a late submission by clicking the checkbox. -- **Editing late reports:** Once you mark a month's report as submitted to the branch office, you won't be able to edit it. However, you can still edit any reports marked as late. -- **Next month's report:** Late reports will automatically be included in the next month's report to the branch office. +* **Marking as Late Report** + If a report comes in after the deadline, simply mark it as a late submission by clicking the checkbox. + +* **Editing Late Reports** + Once you mark a month's report as submitted to the branch office, you won't be able to edit it. However, you can still edit any reports marked as late. + +* **Next Month's Report** + Late reports will automatically be included in the next month's report to the branch office.
Line range hint
82-82: Fix typo in S-1 Report section.There's a typo in the word "sumbitting".
-needed for sumbitting the official report on JW HUB website. +needed for submitting the official report on JW HUB website.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (17)
organized/docs/how-to-use/ministry/img/add-study.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/add-time-popup.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/add-time.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/continuous-goal.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/edit-study.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/hours-goal.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/pause-timer.gifis excluded by!**/*.giforganized/docs/how-to-use/ministry/img/pio-form.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/report-of-pioneer.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/report-of-publisher.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/select-studies.gifis excluded by!**/*.giforganized/docs/how-to-use/ministry/img/service-year-history.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/submit.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/tabs-with-report-months.pngis excluded by!**/*.pngorganized/docs/how-to-use/ministry/img/time-already-in-service.gifis excluded by!**/*.giforganized/docs/how-to-use/ministry/img/time-already-in-service.pngis excluded by!**/*.pngorganized/docs/how-to-use/reports/img/report-states.jpgis excluded by!**/*.jpg
📒 Files selected for processing (1)
organized/docs/how-to-use/reports/field-service-reports.md(1 hunks)
🔇 Additional comments (2)
organized/docs/how-to-use/reports/field-service-reports.md (2)
31-32: LGTM! Clear and consistent status terminology.
The updated status terminology ("Not submitted" and "Pending verification") provides clearer workflow indication and better reflects the actual state of reports in the system.
Line range hint 25-25: Verify image updates and paths.
The PR objectives mention image refresh, but we need to ensure all referenced images are updated and accessible:
- received-reports-bar.png
- report-states.jpg
- report-example.png
Also applies to: 35-35, 52-52



Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
Checklist: