rad app graph allows a -e flag which takes no effect

nithyatsu commented 6 months ago

Steps to reproduce:

Use rad app graph command to query for an application and pass in a nonexistent environment name using the -e flag:

nithya@Nithyas-MacBook-Pro bug7052 % rad app graph corerp-resources-gateway -e doesntexist 
Displaying application: corerp-resources-gateway

Name: http-gtwy-back-ctnr (Applications.Core/containers)
Connections:
  http-gtwy-back-rte (Applications.Core/httpRoutes) -> http-gtwy-back-ctnr
Resources:
:

Observed behavior

The -e flag has no effect on the command.

Desired behavior

The -e flag gets taken under consideration for the command:

query for the application in the specified environment
if the application does not exist in the environment, show error to user ("The application was not found in the specified environment", or something similar)
if the environment does not exist, show error to user ("The specified environment was not found", or something similar)

AB#11781

sylvainsf commented 6 months ago

Agreed this should be covered properly.

radius-triage-bot[bot] commented 6 months ago

:+1: We've reviewed this issue and have agreed to add it to our backlog. Please subscribe to this issue for notifications, we'll provide updates when we pick it up.

We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

radius-triage-bot[bot] commented 3 days ago

This issue is a great one to pickup for new contributors. It should only require small changes and not assume a deep knowledge of the Radius architecture.

We always welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.

For more information on our triage process please visit our triage overview

radius-project / radius