Simplify description from credentials PR #16

ManageIQ / manageiq-providers-embedded_terraform

ManageIQ plugin for the Embedded Terraform provider.

Apache License 2.0

0 stars 10 forks source link

Simplify description from credentials PR #16 #21

Closed jrafanie closed 4 months ago

jrafanie commented 4 months ago

This PR simplifies a few descriptions from #16

Updated screens:

miq-bot commented 4 months ago

Checked commits https://github.com/jrafanie/manageiq-providers-embedded_terraform/compare/0894742d1ff0026c197dfa961f2b741b288ec08f~...35623774ff9307453206ec3460dd988ad7dcf96b with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 2 files checked, 0 offenses detected Everything looks fine. :cake:

jrafanie commented 4 months ago

NOTE: I'll split off the IBM Cloud credentials into a followup PR with two types: cloud and classic infrastructure.

Note, for the IBM cloud, it looks like we need either the cloud API key or both the classic infra user name and classic infra API key so it doesn't make sense to require the cloud API key. Is this correct @putmanoj?

jrafanie commented 4 months ago

This should be ready to go