Feature/librarybooksrenewfeature 2353#13
Open
saicharankamble-ctrl wants to merge 8 commits intofrappe:masterfrom
Open
Feature/librarybooksrenewfeature 2353#13saicharankamble-ctrl wants to merge 8 commits intofrappe:masterfrom
saicharankamble-ctrl wants to merge 8 commits intofrappe:masterfrom
Conversation
… and validation ✨ New Features: - Auto-populate student details (email, class, branch) on student selection - Auto-populate book details from ISBN/Accession Number with cross-population - Auto-calculate issue date (current) and return date (7 days from issue) - Calculate reading period and due days automatically - Real-time stock validation to prevent out-of-stock book issuance - Take-home permission validation for book eligibility - Smart student library books integration with child table sync 🔧 Enhancements: - Prevent duplicate entries in student library records - Update student book count to show only READING status books - Inventory management with automatic quantity reduction - Consolidated messaging system to prevent spam notifications - Enhanced form validation with user-friendly error messages 🐛 Bug Fixes: - Fixed double population issue when ISBN auto-fills accession number - Fixed duplicate entries in student custom_library_books table - Fixed incorrect book counting (now counts only active reading books) - Fixed repeated success messages during form operations - Fixed stock validation edge cases 🎯 Technical Improvements: - Added comprehensive server-side validation hooks - Implemented smart client-side field population - Added proper error handling and user feedback - Optimized database queries for better performance - Added scheduled job for overdue book tracking Files modified: - library_transactions.py: Complete server-side logic implementation - library_transactions.js: Enhanced client-side auto-population - library_transactions.json: DocType structure updates - library_books_student_table.json: Child table enhancements
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix: set renewal datetime when book status is renewed [PR - 13 ] [Issue: 2353]
#13
fix(library): handle book renewal and prevent status-related errors