-
Notifications
You must be signed in to change notification settings - Fork 1
[NEC_NT] Bug fixes #219
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
base: develop
Are you sure you want to change the base?
[NEC_NT] Bug fixes #219
Conversation
… that don't need adapters to be clicked
…able service details screen shows
…ing and formatting across screens that uses date and time
❌ Unit tests failed! @MichaelReyes, please review and fix the issues in the unit tests. Test results are available under the "Artifacts" section of this run in GitHub Actions. |
❌ Unit tests failed! @MichaelReyes, please review and fix the issues in the unit tests. Test results are available under the "Artifacts" section of this run in GitHub Actions. |
✅ Unit tests passed successfully! Test results are available under the "Artifacts" section of this run in GitHub Actions. Please ensure the code is reviewed before proceeding with the merge. |
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.
No changes will be requested, left a few nitpick comments.
|
||
when { | ||
daysDiff == 0 -> { | ||
val timePattern = SystemTimeFormatManager.getTimeFormatPattern() |
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.
Can be extracted to only be calculated once.
} | ||
} | ||
} catch (e: Exception) { | ||
e.printThrowableStackTrace() |
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.
Can be replaced by Timber or Log
@@ -70,5 +73,26 @@ data class GenericListItem( | |||
|
|||
return emptyList() | |||
} | |||
|
|||
private fun formatTime(context: Context, timeInMillis: Long): String { |
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.
Some identical logics can be extracted.
Pull Request (PR) Checklist
Thank you for your contribution! Please confirm that you've checked all the boxes below before submitting your PR. Use
[x]
to check a box, e.g.,[x]
, and make sure there's no space around the brackets.PR Context
Changes
Describe your changes in detail, highlighting the problem it solves or the feature it adds.
Checklist for Reviewers
Documentation and Code Quality
Testing and Reliability
Error Handling and Logging
Testing Procedure
If applicable, provide steps or commands for testing your changes. This can help reviewers and testers.
Work-in-Progress (WIP)
List any remaining work or areas that need additional focus. This section can be updated as the work progresses.
Remember to keep this template updated based on the feedback and evolving project standards.