maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Put Algolia frontend behind feature flag (#20935)

Browse source
This commit is contained in:
Ben Halpern 2024-05-09 09:22:15 -04:00 • committed by GitHub
parent 2ee84c6452
commit 34df26c847
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/javascript/packs/searchParams.js
View file

@ -216,17 +216,12 @@ function search(query, filters, sortBy, sortDirection) {
}
function algoliaSearch(searchParams) {
console.log('Algolia is a work in progress. You may want to turn it off if you see this message.') /* eslint-disable-line */
// console.log(searchParams) /* eslint-disable-line */
// const queryParams = getQueryParams(searchParams);
const paramsObj = getQueryParams(searchParams);
const env = document.querySelector('meta[name="environment"]').content;
const {algoliaId, algoliaSearchKey} = document.body.dataset;
console.log(paramsObj) /* eslint-disable-line */
const client = algoliasearch(algoliaId, algoliaSearchKey);
const indexName = paramsObj.sort_by ? `${paramsObj.class_name || 'Article'}_timestamp_${paramsObj.sort_direction}_${env}` : `${paramsObj.class_name || 'Article'}_${env}`;
const index = client.initIndex(indexName); // Hardcoded to user for now
console.log(index) /* eslint-disable-line */
// This is where we will add the functionality to get search results directly from index with client:
index
.search(paramsObj.search_fields, {
@ -234,8 +229,6 @@ function algoliaSearch(searchParams) {
page: paramsObj.page,
})
.then(({ hits }) => {
console.log('Algolia search results:') /* eslint-disable-line */
console.log(hits); /* eslint-disable-line */
const resultDivs = [];
const currentUser = userData();
const currentUserId = currentUser && currentUser.id;

4
app/views/layouts/application.html.erb
View file

@ -59,8 +59,8 @@
data-ga-tracking="<%= j(Settings::General.ga_tracking_id) %>"
data-cookie-banner-user-context="<%= j(Settings::General.cookie_banner_user_context) %>"
data-cookie-banner-platform-context="<%= j(Settings::General.coolie_banner_platform_context) %>"
data-algolia-id="<%= j(Settings::General.algolia_application_id) %>"
data-algolia-search-key="<%= j(Settings::General.algolia_search_only_api_key) %>"
data-algolia-id="<%= j(Settings::General.algolia_application_id) if feature_flag_enabled?(:algolia_frontend) %>"
data-algolia-search-key="<%= j(Settings::General.algolia_search_only_api_key) if feature_flag_enabled?(:algolia_frontend) %>"
data-algolia-display="<%= j(Settings::General.display_algolia_branding) %>"
data-ga4-tracking-id="<%= j(Settings::General.ga_analytics_4_id) %>">
<%# Repeat of stylesheets in <head> to fix rendering glitch: https://github.com/forem/forem/issues/12377 %>