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
[NTV-174 & NTV-175]: fetch faq & environmentCommitment from GraphQL #1400
Conversation
- call to deprecated methods updated
…kstarter/android-oss into imartin/ntv-191-rewards-missing-fields * 'feature/NTV-181-project-from-graphql' of github.com:kickstarter/android-oss: NTV-182: missing values v1 Currency + No Reward (#1391)
…kstarter/android-oss into imartin/ntv-173-faq-environment-risks * 'feature/NTV-181-project-from-graphql' of github.com:kickstarter/android-oss: NTV-191: rewards missing fields (#1393)
- model - tranformer method - tests
Codecov Report
@@ Coverage Diff @@
## master #1400 +/- ##
=========================================
Coverage 72.22% 72.22%
Complexity 739 739
=========================================
Files 211 211
Lines 6071 6071
Branches 391 391
=========================================
Hits 4385 4385
Misses 1549 1549
Partials 137 137
Continue to review full report at Codecov.
|
@@ -1169,11 +1158,13 @@ private fun projectTransformer(projectFragment: FullProject?): Project { | |||
.staticUsdRate(staticUSDRate) | |||
.usdExchangeRate(usdExchangeRate) | |||
.updatedAt(updatedAt) | |||
// .unreadMessagesCount() TODO: unread messages can be fetched form the User Object | |||
// .unseenActivityCount() TODO: unseen activity can be fetched form the User Object | |||
// .unreadMessagesCount() unread messages can be fetched form the User Object |
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.
Nit: Do we need this commented out code? If not we should remove it, unless it's providing context.
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.
It's for providing context, as those fields have value when fetched from V1, but on GraphQL they do not exist under Project
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.
Awesome! Lgtm! 🚀
📲 What
🛠 How
fullProject
fragmentProjectFaq
new model createdEnvironmentalCommitment
new model createdProject.java
model has two new fields :projectFaqs: List<ProjectFaq>, .envCommitments: List<EnvironmentalCommitment>
👀 See
| | |
📋 QA
ℹ️ We should start moving all the transformer private methods inside
KSApolloClient
into the new fileGraphQLTransformers.kt
to be tested 😄 let me know if any concern/doubts on itProjectViewModel
on the currentproject observable and map the two new fields :)Story 📖
NTV-174
NTV-175