Skip to content

Conversation

@jeremysteerio
Copy link
Contributor

@jeremysteerio jeremysteerio commented Nov 27, 2025

UHF-12520

What was done

  • Add possibility to override elastic endpoint with ELASTIC_DEV_URL option
  • Use EnvironmentResolver for getting elastic url instead of using .env variable
  • Collapse search results for unique hits
  • Change code to reflect backend changes
  • Touch up some ts, biome stuff

How to install

  • Make sure your instance is up and running latest version of dev-branch
    • git checkout dev && git pull origin dev
    • make fresh
  • Run drush en helfi_strategia -y
  • Switch to feature branch
    • git fetch && git checkout UHF-12520
  • Run code updates
    • composer install
    • make drush-deploy drush-locale-update drush-cr

Have your Etusivu instance up and running on the latest dev branch. Make sure helfi_hyte_search module is installed there.
Boot up Sote or Kasko instance as well. Do this:

  • Make sure helfi_hyte_search is installed and you're on latest dev branch
  • Run drush sapi-c hyte; drush sapi-rt hyte; drush sapi-i hyte

How to test

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 65%)

See analysis details on SonarQube Cloud

content={units.length.toString()}
icon={<IconLocation />}
icon={<span className='hel-icon hel-icon--location' />}
label={`${Drupal.t('Locations', {}, { context: 'Hyte search' })}`}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one needs translation 🇫🇮 🇸🇪

Copy link
Contributor

@teroelonen teroelonen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the one small translation addition this is good to go 🦖

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.

3 participants