-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add -y
option for bulk-add-officers
to bypass user prompt
#222
Comments
Can you please help me out how would I implement it as a cli? |
Hi @sonali-rajput, please go ahead! I'll assign it to you 😄 |
Okay thanks for your patience @sonali-rajput - what we'll need to do is add another option to the command here which is just a boolean flag for skipping the check prompt: OpenOversight/OpenOversight/app/commands.py Lines 438 to 452 in c7ce36c
Then in the check step mentioned in the issue description, we also add another condition for seeing if that command line argument was passed in, and skip the prompt yes/no. This would look like: if current_app.config["ENV"] == "testing" or bypass_prompt or prompt_yes_no(
"Do you want to commit the above changes?"
): You can also use the |
Similar to 221, it would be nice to be able to bypass the user prompt when performing the bulk upload that's part of
just fresh-start
. This is already possible if the end is set to "testing", but the CLI argument would be useful too:OpenOversight/OpenOversight/app/commands.py
Line 534 in c7ce36c
The text was updated successfully, but these errors were encountered: