You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
AI-Generated Summary: This pull request contains refactoring changes in the GLASSDataSubmissionDefaultRepository.ts file. The filter values are extracted into their own constants to make the code clearer and more readable. The logical operations inside if-statements are also streamlined and refactored to decrease code complexity. In addition, the conditional statements are revised to introduce consistency and further improve readability. The changes are all structural and do not affect the functionality of the code.
AI-Generated Summary: This pull request refactors parts of the GLASSDataSubmissionDefaultRepository.ts file for better readability and maintainability. The conditional logic within the row filtering functions has been extracted into separate variables, providing clearer expressions of the filtering criteria. This change primarily affects the conditionals that were previously within filter method callbacks. Now, instead of conducting checks within these callbacks, the checks are assigned to their respective variables which makes the code easier to understand, debug, and maintain.
The reason will be displayed to describe this comment to others. Learn more.
Thanks @deeonwuli !! Now all conditionals are read much better than before 🎉
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
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.
📌 References
📝 Implementation
📹 Screenshots/Screen capture
🔥 Notes to the tester
REACT_APP_REPORT_VARIANT=glass-submissionREACT_APP_DHIS2_BASE_URL=https://dev.eyeseetea.com/who-dev-238/