RedHatInsights / sources-ui

Static deploy repo: https://github.com/RedHatInsights/sources-ui-deploy
Apache License 2.0
5 stars 47 forks source link

Update validation to avoid issues with azure app id #1213

Closed CodyWMitchell closed 4 months ago

CodyWMitchell commented 4 months ago

Update validation to avoid issues with azure app id

Add validation to require that Azure subscription ID must only contain letters, numbers, and hyphens

RHCLOUD-32801

Checklist ☑️

codecov-commenter commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.75%. Comparing base (2c29433) to head (1c9cd04).

:exclamation: Current head 1c9cd04 differs from pull request most recent head 5342ec2

Please upload reports for the commit 5342ec2 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1213 +/- ## ======================================= Coverage 97.75% 97.75% ======================================= Files 184 184 Lines 3702 3702 Branches 1053 1053 ======================================= Hits 3619 3619 Misses 77 77 Partials 6 6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.