Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

add ui state in view models for Compose #862

Open
qamarelsafadi opened this issue Jun 19, 2023 · 2 comments
Open

add ui state in view models for Compose #862

qamarelsafadi opened this issue Jun 19, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@qamarelsafadi
Copy link
Contributor

qamarelsafadi commented Jun 19, 2023

In Gallery Screen, I believe that the screen would have a 3 states

The first one is when the request is already successful and the data is showing,

Second, when the data is loading we might have to show a loader to tell the user that the data is on its way to be shown

The third one is if the request has failed we need to let the user know that there is something wrong with the API or his internet connection.

Part of GTC open source initiative

@arriolac
Copy link
Contributor

Please add more details in the description on why this should be done. I'm not convinced this is a necessary change.

@qamarelsafadi
Copy link
Contributor Author

Done,

Let me know what do you think.

@arriolac arriolac added the enhancement New feature or request label Dec 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants
@arriolac @qamarelsafadi and others