-
-
Notifications
You must be signed in to change notification settings - Fork 339
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
Replacing Cupboard with Room #29
Conversation
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.
I've finished :-)
Don't hesitate to ask or to say no if I said something wrong or weird.
library/src/main/java/com/readystatesoftware/chuck/api/ChuckCollector.java
Outdated
Show resolved
Hide resolved
} | ||
|
||
val requestDateString: String? | ||
get() = requestDate?.toString() |
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.
This class is an entity, it shouldn't be in charge of formatting values. Same for responseDateString
, durationString
, etc.
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.
So this can easily be removed. The problem is unfortunately wider.
So far I tried to make sure HttpTransaction.kt
had a similar api with HttpTransaction.java
.
I'll say we can probably refactor all this stuff outside of the data class and have it inside say FormatUtils?
I'll anyway keep those refactorings outside of this PR.
library/src/main/java/com/readystatesoftware/chuck/internal/data/entity/HttpTransaction.kt
Show resolved
Hide resolved
library/src/main/java/com/readystatesoftware/chuck/internal/data/entity/HttpTransaction.kt
Outdated
Show resolved
Hide resolved
library/src/main/java/com/readystatesoftware/chuck/internal/data/room/RecordedThrowableDao.kt
Outdated
Show resolved
Hide resolved
library/src/main/java/com/readystatesoftware/chuck/internal/data/room/RecordedThrowableDao.kt
Outdated
Show resolved
Hide resolved
library/src/main/java/com/readystatesoftware/chuck/internal/data/room/RecordedThrowableDao.kt
Outdated
Show resolved
Hide resolved
.../src/main/java/com/readystatesoftware/chuck/internal/ui/transaction/TransactionActivity.java
Outdated
Show resolved
Hide resolved
.../main/java/com/readystatesoftware/chuck/internal/ui/transaction/TransactionListFragment.java
Outdated
Show resolved
Hide resolved
Hey @olivierperez so I've addressed all your comments + left a couple of questions. |
Took some time but I managed to remove Cupboard completely :)
Unfortunately the Pull Request is pretty big :( Let me know if I should split it somehow to make it easier to review.
Summary of changes: