Command can be run via ./manage.py clean_outgoing_urls [--dry-run]
dry-run does a test run, and does not save the cleaned results
Stats are printed in task logger per each chunked run
Note: Stats may not be correct on the second run through if REDIRECT_URL is falsey.
Empty'd out the REDIRECT_URL strings for each conf (Any additional REDIRECT_URL is for tests)
There was an upstream fix, but it wouldn't really work since we want to remove the url, not change the url. Easier to just re-clean them. We can adjust the fields we want to clean, and I'm also not fixed on the stats, so we can nix those too if you'd like.
I ran this against 2000~ test addons locally, and the tasks run pretty much instantly. (Granted both my test machines are pretty beefy.) This needs a staging qa check, and we'll need to empty/none out any lingering REDIRECT_URL settings before running this.
Also I'm not impressed with whoever decided not to have a matching meta_type or type field on the translations table. :smile:
Fixes #203
./manage.py clean_outgoing_urls [--dry-run]
dry-run
does a test run, and does not save the cleaned resultsThere was an upstream fix, but it wouldn't really work since we want to remove the url, not change the url. Easier to just re-clean them. We can adjust the fields we want to clean, and I'm also not fixed on the stats, so we can nix those too if you'd like.
I ran this against 2000~ test addons locally, and the tasks run pretty much instantly. (Granted both my test machines are pretty beefy.) This needs a staging qa check, and we'll need to empty/none out any lingering
REDIRECT_URL
settings before running this.Also I'm not impressed with whoever decided not to have a matching
meta_type
ortype
field on the translations table. :smile: