Added dropdown for fqdn and ip address.

[himanshi-chhabra]

Signed-off-by: Himanshi Chhabra hchhabra@progress.com

🔩 Description: What code changed, and why?

We currently require both the FQDN and the IP address of the chef server. We need to change and make it optional so add a dropdown in which the user can select any option of FQDN or IP address

⛓️ Related Resources

#4945 #4998

👍 Definition of Done

Added a dropdown in which the user can select any option of FQDN or IP address

👟 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

To add data https://github.com/chef/automate/blob/master/dev-docs/adding-data/adding_test_data.md#adding-data-to-infra-views
Steps for policy file tab:

1. Go To Infrastructure -> Chef Servers -> click Add chef server -> Add name -> Select FQDN or IP Address option -> Add FQDN or IP Adress form filed data -> click Add chef server

✅ 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.

🔨 Explore the source changes: abd52c4

🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-automate/deploys/6144292579b7fe000822df6e

[chaitali-mane]

Space between 'Type' and dropdown must be less.

[chaitali-mane]

If I select IP Adress and I cancel/ close modal and open again then the dropdown value is IP Adress. Same for the FQDN. Can dropdown be set to the default value?
Dropdown width is smaller compared to other

[chaitali-mane]

I have some observations

1. Font weight of dropdown must be normal.
2. space between dropdown and below test should be more.
3. height of dropdown should be the same

[chaitali-mane]

If there is no data available for FQDN or IP Address then can we show '-' instead of empty?

[chaitali-mane]

Working as expected

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication