Hotfixes for breedform and yearlyreport auto herd selection #716
Conversation
This file contains 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
…nagement - Changed initial state of showBirthForm from true to false for better user experience. - Enhanced autoFillBreedDate function to include robust date parsing and validation, ensuring accurate calculation of breed date from birth date. - Updated logic in useEffect to handle cases where breed_date is null or empty, improving the reliability of the form's auto-fill feature. These changes aim to streamline the breeding form functionality and enhance the overall user experience by ensuring accurate date handling.
…tep and SelectHerdStep components - Updated the user role checks to handle optional chaining for manager roles, ensuring more robust access control. - Refactored herd options initialization in SelectHerdStep to streamline the logic for displaying herds based on user roles (admin, manager, owner). - Enhanced the rendering logic to improve user experience, particularly for users with a single herd, by auto-selecting the herd when applicable. - Improved the overall structure and readability of the code by organizing callback functions and effects more clearly. These changes aim to enhance the user experience and maintainability of the components by ensuring accurate role-based access and streamlined herd selection.
- Removed unused imports and cleaned up code by eliminating commented-out sections, enhancing readability. - Streamlined user role checks and herd selection logic to ensure accurate display of herds based on user roles (admin, manager, owner). - Improved rendering logic for breeding tables, including better handling of empty states and user prompts. - Enhanced the dialog for adding new breedings, ensuring a more intuitive user experience. These changes aim to enhance maintainability and user experience within the SelectHerdStep component.
…for improved functionality and user experience - Added optional handleNext prop to SelectGenebankStep to facilitate navigation after genebank selection. - Streamlined user role checks in YearlyReportMultiStepForm, enhancing clarity and ensuring accurate step progression based on user roles (admin, manager). - Implemented auto-selection of genebank for non-admin/non-manager users, improving user experience by reducing manual input. - Removed unnecessary comments and debug logs to enhance code readability and maintainability. These changes aim to improve the overall functionality and user experience within the affected components.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.