Fix/added loader to all pages

[rayy40]

Fixes

Use common loader across all the pages of the application
Reference - #1494

Solution

For the pages that lacked a loading state, the loading state was created, and a 'Loader' was added to display the loading.
Additionally, the pages that were already in a loading state were changed to use the proper 'Loader' style.

import Loader from "common/Loader/index"

<div className="flex h-full w-full items-center justify-center">
      <Loader />
</div>

[rayy40]

Hey @prasanthchaduvula I have raised the PR for the issue #1494
Can you please review it and let me know if I should make any changes?

[akhilgkrishnan]

Thanks @rayy40 for working on this 👍 .
If possible could you please record a demo video for the output.

[rayy40]

@akhilgkrishnan , I have updated the changes.
Can you please take a look at it?

[akhilgkrishnan]

@apoorv1316 Could you please test this PR?

[supriya3105]

@apoorv1316 Have you tested this PR ?

[apoorv1316]

@prasanthchaduvula We have an empty state for reports and a few other pages that appear while the page is loaded and in this PR we are using the 'Miru Loader' on almost all pages. It's not consistent. Let's try to make it more consistent across the app.

[akhilgkrishnan]

@prasanthchaduvula We have an empty state for reports and a few other pages that appear while the page is loaded and in this PR we are using the 'Miru Loader' on almost all pages. It's not consistent. Let's try to make it more consistent across the app.

If possible can you list all the pages?

[rayy40]

@apoorv1316 @akhilgkrishnan , Hey guys any updates?

[akhilgkrishnan]

@apoorv1316 @akhilgkrishnan , Hey guys any updates?

@apoorv1316 We can create a separate ticket adding the loader for missing pages. Could you please check the current implementation, if everything looks good we can merge.