[Frontend]: Add Spinners To All Actionable Buttons

[Mayank0255]

Add spinners to all buttons so that when the user clicks a button, it shows a spinner and the user gets to know that some process is going on

[shravankumar-fs]

Hey Mayank
Could you assign this to me? I'd like to assist on this?

Also can i use this without setting up the backend locally?

[Mayank0255]

@shravankumar-fs
Actually backend would be required, otherwise from where will you be getting all the data and test properly.

Let me know if you are in to take up this issue, will assign it to you then

[Mayank0255]

And this might help for loading buttons,
https://stackoverflow.design/product/components/buttons/#loading

[Mayank0255]

@shravankumar-fs

Found a workaround to avoid setting up backend,
Can just change the url here to this https://stackoverflow-clone-backend.herokuapp.com

Let me know if you are interested and would want me to assign it to you

[shravankumar-fs]

Hi @Mayank0255
Please assign it to me, I'll do.

Also currently which buttons need this feature?

I see some places with no action event being triggered

[Mayank0255]

@shravankumar-fs
Ya sure, go ahead.

So, spinners will be required wherever an api call is going, like when the components of a page are loading, when you click on a button, etc.

Just explore the codebase add spinners wherever you think is required

[Mayank0255]

@shravankumar-fs Any progress on this?

[amreekminocha]

Hi.. is anyone working on this issue..
I would like to work on it if no one else is assigned.

[Mayank0255]

@amreekminocha
Ya sure, go ahead.

Any idea how much time will it take you to complete this task?

[amreekminocha]

Thanks for assigning..
I'll go through the project once then i'll tell you the required time

[mayank-zoman]

Thanks for assigning.. I'll go through the project once then i'll tell you the required time

Sure, not an issue

[amreekminocha]

Hi I am having this error
Stackoverflow-Clone-Frontend/node_modules/webpack-dev-server/lib/Server.js:348
} catch {

Can you help

[Mayank0255]

@amreekminocha

Can you pls send me the complete error? I can't firgure out anything from this.

And can you please let me know that did you get this error while setting up the repo or what?

[amreekminocha]

Yes.. I am getting this error while setting the frontend repo

[amreekminocha]

Hi do share a point of contact as well

[Mayank0255]

Hi do share a point of contact as well

sure, you can reach out to me on discord https://discords.com/bio/p/mayank0255

[Mayank0255]

Yes.. I am getting this error while setting the frontend repo

Can you try installing react-scripts individually and then starting the server?

npm i react-scripts

[Mayank0255]

@amreekminocha and check on which node version are in currently?
It should be above 14, 16 is recommended

[amreekminocha]

This worked.. thanks

[amreekminocha]

Hi Can you share the buttons where the spinner is require

[Mayank0255]

Hi Can you share the buttons where the spinner is require

So, spinners will be required wherever an api call is going, like when the components of a page are loading, when you click on a button, etc.

Just explore the codebase add spinners wherever you think is required

It's ok if you miss some places

[Mayank0255]

@amreekminocha how's the progress on this?