fix: Add preflight checks and improve error handling in the UI

floreks commented 1 year ago

Summary

Added some preflight checks before running plural install UI:
- check if inside a git repo (wkspace.Preflight)
- check if context.yaml exists
- check if workspace.yaml exists
Added global apollo client error handler to display toast with error message.
Fixed an issue with provider name not being normalized for further GQL API usage (this broke UI on GCP provider)
Fixed an issue with grommet fileinput not working properly in production build (worked in dev)
Fixed an issue with configure OIDC not respecting confirm option

Locally

[ ] If required, I have updated the Plural documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have added a meaningful title and summary to convey the impact of this PR to a user.
[ ] I have added relevant labels to this PR to help with categorization for release notes.

michaeljguarino commented 1 year ago

I actually think you just need to wrap the function in the rooted validator like here: https://github.com/pluralsh/plural-cli/blob/main/cmd/plural/plural.go#L91 and it'll provide our standard error message when the command is run outside the repo

floreks commented 1 year ago

Done. I have also wrapped it with tracked method.