upcoming: [M3-7743] - Handle errors gracefully when FF enabled without MSW

[cpathipa]

Description 📝

Handles error scenarios gracefully when MSW is off.

Changes 🔄

List any change relevant to the reviewer.

• Code cleanup AccessKeyTableRow
• Decoupled AccessKeyTableRow
• Decoupled HostNameTableCell

How to test 🧪

Prerequisites

(How to setup test environment)

• Turn on OBJ Multi Cluster FF
• Turn OFF MSW

Reproduction steps

(How to reproduce the issue, if applicable)

• Checkout to develop turn on OBJ Multi Cluster FF and MSW OFF.
• Navigate to Accesskey landing page.
• Should notice app crashing.

Verification steps

(How to verify changes)

• Ensure APP is not crashing when FF is turned on in Object storage flow, like Create Bucket, Create / update Access key, Access key landing page etc.
• Turn off the feature flag and ensure there is no regression in object storage flow.

As an Author I have considered 🤔

Check all that apply

• [ ] 👀 Doing a self review
• [ ] ❔ Our contribution guidelines
• [ ] 🤏 Splitting feature into small PRs
• [ ] ➕ Adding a changeset
• [ ] 🧪 Providing/Improving test coverage
• [ ] 🔐 Removing all sensitive information from the code and PR description
• [ ] 🚩 Using a feature flag to protect the release
• [ ] 👣 Providing comprehensive reproduction steps
• [ ] 📑 Providing or updating our documentation
• [ ] 🕛 Scheduling a pair reviewing session
• [ ] 📱 Providing mobile support
• [ ] ♿ Providing accessibility support

[github-actions[bot]]

Coverage Report: ✅
Base Coverage: 81.33%
Current Coverage: 81.33%

[cpathipa]

AccessKeyTableRows @dwiley-akamai I see what you mean: eliminate AccessKeyTableRows and loop through AccessKeyTableRow within the AccessKeyTableBody, which we could definitely do. I thought we would keep the AccessKeyTableRow logic in AccessKeyTableBody.

[jaalah-akamai]

cc: @hana-linode for review

[cpathipa]

Only issue I see is this when the Multi-Cluster feature flag is on

Now, we return an empty cell instead of null when there is no data(MSW off). This addresses the issue.. ( f5c8be6)