[Integrate] As a user, I can see Trending list on Home page

nimblehq / jetpack-compose-crypto

This is an sample Crypto market prices app built with Jetpack Compose

https://nimblehq.co/

MIT License

21 stars 5 forks source link

[Integrate] As a user, I can see Trending list on Home page #14

Closed hoangnguyen92dn closed 2 years ago

hoangnguyen92dn commented 2 years ago

Why

When the user launches the app, it will show the "Trending 🔥" section on the Home page.

Acceptance Criteria

Mapping the data from API response to display on UI:
- If the field price_change_percentage_24h_in_currency:
- Positive number: Use green color with arrow up #10DC78
- Negative number: Use red color with arrow down #F15950
Use Coil to load the coin image ~~- Handle pagination when user scroll down to end of the list~~

Design

N/A

Resources

N/A

luongvo commented 2 years ago

Handle pagination when user scroll down to end of the list

@hoangnguyen92dn Do you mean we should implement scroll down to load more? 🤔

hoangnguyen92dn commented 2 years ago

@luongvo Yup. It would be a challenge to handle pagination with Column in Compose 😄

luongvo commented 2 years ago

@hoangnguyen92dn I see what you mean but I think we should separate this AC into a new ticket as the integration part is quite big enough from this PR https://github.com/nimblehq/jetpack-compose-crypto/pull/31. You can create a new ticket and assign it to me 🧑‍🍳

hoangnguyen92dn commented 2 years ago

@luongvo That's fine. I will create anothet ticket for pagination. Thank you.

hoangnguyen92dn commented 2 years ago

@luongvo I updated the description to get rid of pagination on this ticket and created a new one https://github.com/nimblehq/jetpack-compose-crypto/issues/32 to handle that.