Hide "Asset Type"

[shyamprakash123]

WHAT

🤖[deprecated] Generated by Copilot at be3b8e2

This pull request removes the asset type feature from the frontend components that deal with assets. The asset type is no longer a relevant attribute for filtering, creating, or managing assets, since the backend API only supports internal assets. The affected components are AssetFilter, AssetCreate, and AssetManage.

Proposed Changes

• Fixes Hide "Asset Type" (Assets to be always "Internal") #6789

• Removed "Asset Type" select input from asset create/edit form.

• Upon creating an asset, it is always created with asset type as "Internal" (default)

• Removed Asset Type label from the asset details page.

• Removed Asset Type filter from Advanced Filters also.

@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

HOW

🤖[deprecated] Generated by Copilot at be3b8e2

• Remove the asset type filter option from the AssetFilter component and the filter logic (link, link, link, link)
• Remove the asset type input option from the AssetCreate component and the asset creation logic (link, link, link, link, link, link)
• Remove the asset type display from the AssetManage component (link)
• Hardcode the asset type value to AssetType.INTERNAL for creating or editing the assets in the AssetCreate component (link)

[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 17, 2024 3:23pm

[netlify]

✅ Deploy Preview for care-egov-staging ready!

Name	Link
🔨 Latest commit	c4b59a4
🔍 Latest deploy log	https://app.netlify.com/sites/care-egov-staging/deploys/65a7f0b1c8b64c0008f2fe5b
😎 Deploy Preview	https://deploy-preview-6835--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.

[nihal467]

@shyamprakash123

• modify the cypress test in asset file, when you are hiding the asset type, make necessary change in the file as well

[shyamprakash123]

@nihal467

Cypress Tests are now fixed.

[nihal467]

@shyamprakash123 fix the failing cypress test, you should be fixing cypress with running care in the local frontend and backend by loading dummy data

[shyamprakash123]

@nihal467 I reverted the cypress test variables, can you re-test this?

[rithviknishad]

The cypress test still seems to be failing, could you look into it?

[nihal467]

@shyamprakash123 fix the cypress issue

[github-actions]

Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions.

[github-actions]

Hi, @gigincg, @nihal467, @khavinshankar, @mathew-alex, This pr has been automatically closed because it has not had any recent activity. Thank you for your contributions. Feel free to repopen the pr.

[github-actions]

👋 Hi, @shyamprakash123,
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.