-
Notifications
You must be signed in to change notification settings - Fork 479
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hide "Asset Type" (Assets to be always "Internal") #7061
Hide "Asset Type" (Assets to be always "Internal") #7061
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
👋 Hi, @shyamprakash123, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
a5adf30
to
f95310e
Compare
👋 Hi, @shyamprakash123, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
👋 Hi, @shyamprakash123, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
eab1f63
to
e7299ff
Compare
ba397ed
to
88b5444
Compare
@shyamprakash123 you might have mishandled the merge conflicts and the file changes are not related to the PR. I would suggest taking advice on how to handle merge conflicts and for now, close this PR and create a new one. |
Proposed Changes
Fixes Hide "Asset Type" (Assets to be always "Internal") #6789
Remove "Asset Type" select input from asset create/edit form.
Upon creating an asset, it should always create with asset type as "Internal"
Remove it from the asset details page too.
Remove Asset Type filter from Advanced Filters
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist