Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cli): add feature to make context a required field #266

Merged
merged 7 commits into from
Apr 4, 2024

Conversation

joe-prosser
Copy link
Contributor

@joe-prosser joe-prosser commented Apr 3, 2024

Makes the re -c <context> required if you're a UiPath user / if you opt for that in the config

@joe-prosser joe-prosser self-assigned this Apr 3, 2024
Copy link
Collaborator

@tommilligan tommilligan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG, thanks!

Ok(client)
}

const DOMAINS_THAT_REQUIRE_CONTEXT: [&str; 2] = ["uipath.com", "reinfer.io"];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would vote let's make it required for everyone until they opt out by setting the config flag honestly, I agree with you about the bad implications of uploading to the wrong place. I think quite common for even client orgs to have a staging and prod tenant right?

Copy link
Contributor Author

@joe-prosser joe-prosser Apr 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah ok cool - did worry about that breaking any existing scripts that people have. But maybe forcing them to opt out isn't a bad thing

CHANGELOG.md Outdated
Comment on lines 1 to 4
# Unreleased
- Add option to make context option a required field
- Make context option a required field for internal users

Copy link
Collaborator

@tommilligan tommilligan Apr 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should note explicitly that this is a breaking change for reinfer/uipath users.

@joe-prosser joe-prosser merged commit fb095d7 into master Apr 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants