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

Adds utility scripts for new city deployments and revealing/hiding neighborhoods #3751

Merged
merged 10 commits into from
Nov 22, 2024

Conversation

misaugstad
Copy link
Member

Resolves #3748

Adds utility scripts for some of the most common things that I have to do: launching new cities and showing/hiding neighborhoods within a city, and I made a script for each of them. For the latter, I generalized it so that it can be run either locally or on the server since there were a few things to do differently between the two. I also moved all of our db scripts into a new scripts dir to deal with the clutter!

All of the stuff that I automated here was pretty simple to do at this point, having done each of them over a dozen times. And I had really nice documentation for how to do it. It was just a lot of manual copying and pasting, verifying that output was correct, etc., which made it easy to script.

It was easy to do, but there were just enough points where I could do something wrong when copying/pasting that I had to pay very close attention. Using these scripts will certainly speed up these processes in the future, and it should also significantly reduce my stress when doing it! I'm always worried that I missed a step, and am double checking output at multiple points throughout the process, because the mistakes can cascade and get worse. Now I will be able to just run the script, check the output, and move on!

Before/After screenshots (if applicable)

Here's just a quick screenshot of the terminal output when I did a test run of hiding a neighborhood on a test server and then revealing it again. The script prompts me for a bunch of info, then moves forward!
Screenshot from 2024-11-22 16-18-24

Things to check before submitting the PR
  • I've written a descriptive PR title.
  • I've added/updated comments for large or confusing blocks of code.
  • I've included before/after screenshots above.

@misaugstad misaugstad self-assigned this Nov 22, 2024
@misaugstad misaugstad merged commit 49b5eec into develop Nov 22, 2024
@misaugstad misaugstad deleted the 3748-utility-scripts-for-new-city-deployments branch November 22, 2024 23:49
@misaugstad misaugstad mentioned this pull request Nov 23, 2024
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.

Add more automation to deploying new cities, adding/hiding neighborhoods, etc
1 participant