Fixed loading still active & not showing the message if admin key is invalid.

[vinay033]

Signed-off-by: Vinay Sharma vsharma@chef.io

🔩 Description: What code changed, and why?

Any error while loading roles, environments, data bags, etc. loading still active. For cookbooks, it's working fine.
In case of any type of error response, it's always showing the message on the page.

⛓️ Related Resources

#4825

👍 Definition of Done

I have added some changes to fix this issue.

👟 How to Build and Test the Change

STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command 

npm run serve:hab

navigate to `https://a2-dev.test/infrastructure/chef-servers`
add some sample data and then test the changes using tab change

Go to the `infrastructure tab` from the top navigation bar then you can see the `Chef Server`
you need to add the server first ---> click to this server, then you need to add the org with an invalid admin key.
Note: Add an invalid admin key at the time of adding  the org
 
--> click to org ---> you can see the multiple tabs --> cookbooks, roles, environments, databags, clients 
if you click on any of the tabs then you can see the message.

✅ Checklist

All PRs must tick these:

• I have read the CONTRIBUTING document.
• All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

• Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
• Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
• Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
• Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
• Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

• Tests added/updated? (all new code needs new tests)
• Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

📷 Screenshots, if applicable

[netlify]

Deploy preview for chef-automate processing.

Building with commit 5c6175b

https://app.netlify.com/sites/chef-automate/deploys/607c7f66736ccf0007a4b037

[vivekshankar1]

Thanks @vinay033 , Its working as expected

[chaitali-mane]

Thanks, @vinay033 , Looks good to me