Show a spinner when the API request for Star an repo is running about mobile-app

[notbrood]

Can I start workiing on this?

[tanmoysrt]

Can I start workiing on this?

cc : @vedant-jain03

[notbrood]

Hey @Tanmoy741127, do you think it would be UI friendly if there was no spinner blocking the view? I have added a bool function which checks if the request is finished, and then only allows the user to star the project again, what are your views on that?

[tanmoysrt]

@notbrood
That will work. Try out how it will look,
if you can add a little spinner or pulse type of loader in the box of the star icon

[notbrood]

@Tanmoy741127
Just wanted to let you know that I'm not very comfortable with the get state management aspect of our project. I could try working on it, but I don't think I have the necessary skills to do it properly.

Regarding the bool function, I'm happy to create a PR for the function, and let someone else add the spinner.

[tanmoysrt]

@notbrood in the mobile-app we are using Provider for state management solution and GetX is using for different purposes. So you can give it a try.

[notbrood]

Okay, I'll give it a try!

[notbrood]

I was able to, It looks UI Friendly as well!

[notbrood]

2023-03-08.03-56-08.mp4

Here's how it looks! What are your views?

[tanmoysrt]

Here's how it looks! What are your views?

Great ! Try to make the size same as the star icon

[notbrood]

I did, how does this look?

2023-03-08.13-28-40.mp4

[tanmoysrt]

I did, how does this look?

It's looking good now !

[notbrood]

Okay, I'm going to make a PR for this now!

[tanmoysrt]

Okay, I'm going to make a PR for this now!

I did, how does this look?

cc @manjotsidhu @aman-singh7