Show Asset Downtime on assets lists page

[Pranshu1902]

Proposed Changes

• Fixes Show "Asset Down" in asset list if asset's last uptime status is down #6819
  Show Asset Downtime on the assets lists page if last uptime status was "down"
  

Backend PR: #1794
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

• Add specs that demonstrate bug / test a new feature.
• Update product documentation.
• Ensure that UI text is kept in I18n files.
• Prep screenshot or demo video for changelog entry, and attach it to issue.
• Request for Peer Reviews
• Completion of QA

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
care-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 22, 2024 3:15pm

[netlify]

✅ Deploy Preview for care-egov-staging ready!

Name	Link
🔨 Latest commit	57926fe
🔍 Latest deploy log	https://app.netlify.com/sites/care-egov-staging/deploys/65ae864e59eb840008c81465
😎 Deploy Preview	https://deploy-preview-6952--care-egov-staging.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[rithviknishad]

I don't think this is the right way to do it. The asset list page now fetches paginated assets in one call, and paginated availability records in another call.

This does not ensure that you have the availability records of all the assets of the current assets in view.

I think the solution is to add a serializer-level field in the asset list API's serializer that says whether it is down or not.

cc: @Ashesh3

[rithviknishad]

Also, some other icon would be better. A setting cog icon does not suit for indicating downtime.

[Ashesh3]

I don't think this is the right way to do it. The asset list page now fetches paginated assets in one call, and paginated availability records in another call.

This does not ensure that you have the availability records of all the assets of the current assets in view.

I think the solution is to add a serializer-level field in the asset list API's serializer that says whether it is down or not.

cc: @Ashesh3

I agree with you here.

I think the solution is to add a serializer-level field in the asset list API's serializer that says whether it is down or not.

^ this is the ideal solution

[github-actions]

👋 Hi, @Pranshu1902,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.

---

This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

[Pranshu1902]

@rithviknishad is this icon ok?

[nihal467]

@Pranshu1902 the backend review is still going on, posting the test to next week

CC: @rithviknishad @Ashesh3 @sainak

[rithviknishad]

LGTM

[Pranshu1902]

@nihal467 @khavinshankar the backend PR has been merged, this PR is good to merge

[nihal467]

LGTM

[github-actions]

@Pranshu1902 We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.