-
Notifications
You must be signed in to change notification settings - Fork 36
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
Feature/bulk reindex urls #175
Conversation
@peggles2 , I thought you were going to make all the re-index changes on one branch? This PR duplicates the db/structure.sql changes made in #172, but lacks the specs for the column addition. I think it would be best to wait for #172 to be merged to master, and then open a PR with only the additive changes so that we don't have two PRs open with the same db change. |
Adding a Do Not Merge label to this, as it's dependent on the migration in #172 |
@peggles2 , there are some conflicts in the VCR cassettes. I recommend checking out those cassettes from master, and then re-recording them to update them:
|
fbd20e2
to
4ecf352
Compare
Almost there - just a few stylistic & indendation changes to make |
…e_affiliate_has_the_jobs_govbox_enabled.yml
* SRCH-179 - Add a boolean column to the searchgov_urls * update structure.sql * migration file for enqueued for reindex to searachgov_url * update migration, update test cases * typo * remove extra space * migrated database
remove parenthesis in rspec test
8f531c8
to
4170e37
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🎊
SRCH-183 - Update the SearchgovUrl#fetch method to set that boolean to false
SRCH-181 - Update the SearchgovURL.fetch_required scope to look for records where that boolean is true