fix: apply Algolia settings regardless of clear_index value#31
Open
PedroAntunesCosta wants to merge 1 commit into
Open
fix: apply Algolia settings regardless of clear_index value#31PedroAntunesCosta wants to merge 1 commit into
PedroAntunesCosta wants to merge 1 commit into
Conversation
Previously, when clear_index was false, the settings (including attributesForFaceting) from custom_settings in config JSON were not applied, causing index configuration to be lost. This fix ensures set_settings() is called in both branches to maintain consistent index configuration.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes a critical bug where Algolia index settings (including
attributesForFaceting) were not being applied whenclear_index: false, causing configuration to be reset on every scrape when new documentation is added.Problem
When
clear_indexis set tofalsein the config, the scraper copies the production index to a temporary index but skips applying the settings from the config JSON. This means:custom_settingsin config files were ignoredattributesForFacetingwould get lostSolution
Modified
scraper/src/algolia_helper.pyto callset_settings()in both branches (whenclear_indexis true OR false), ensuring configuration from JSON files is always applied consistently.Changes
scraper/src/algolia_helper.pyself.algolia_index_tmp.set_settings(settings)to theelseblock (line 36)clear_indexvalueImpact
attributesForFacetingand other custom settings will be properly maintained