Skip to content
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

UHF-9156: Update HDS to v3 #861

Merged
merged 18 commits into from
Jan 19, 2024
Merged

UHF-9156: Update HDS to v3 #861

merged 18 commits into from
Jan 19, 2024

Conversation

Jussiles
Copy link
Contributor

@Jussiles Jussiles commented Dec 21, 2023

UHF-9156

What was done

As fas as I know HDS package changes should only affect our React searches. HDS styles we use are manually copied to HDBT and locked to certain version because of that.

  • Update HDS packages to version 3 and HDS dependencies
  • Stop using HDS Tabs in School search. Instead use our own code
  • Fix use of html-parser in Ploughing search
  • Update other React/js packages that had dependapot PR:s open
  • Update node version to 18.19.0
  • Remove Joi, es6-promise, Buffer, path-browserify packages because those seemed unused
  • Add npm audit fix fixes

How to install

  • Make sure these instance is up and running on latest dev branch:
  • Etusivu, Rekry, KYMP, KASKO, SOTE
    • git pull origin dev
    • make fresh
  • Update the HDBT theme
    • composer require drupal/hdbt:dev-UHF-9156-update-hds-to-v3
  • Run make shell
  • Run drush sapi-rt; drush sapi-c; drush sapi-i; drush cr

How to test

Check also running the NPM commands with some Drupal instance.

  • Install HDBT: composer reinstall drupal/helfi_platform_config --prefer-source
  • Go to the root of HDBT theme
  • Install it: nvm use; npm i;
  • Check that running npm "scripts" from package.json file work. You may need to add some code changes to check all the command e.g. linting.

@Jussiles Jussiles changed the title WIP: UHF-9156: Update hds to v3 WIP: UHF-9156: Update HDS to v3 Dec 21, 2023
@Jussiles Jussiles changed the title WIP: UHF-9156: Update HDS to v3 UHF-9156: Update HDS to v3 Dec 27, 2023
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.

This one worked but there was one quite significant issue with most (not all 🤔) of the focus rings on filter elements having changed into something barely noticeable. The border of the element changes to black from gray but the change is really small so its really hard to see where your focus is. This needs to be checked before this can be merged. Otherwise everything worked perfectly!

@j-mys
Copy link
Contributor

j-mys commented Jan 8, 2024

This one worked but there was one quite significant issue with most (not all 🤔) of the focus rings on filter elements having changed into something barely noticeable. The border of the element changes to black from gray but the change is really small so its really hard to see where your focus is. This needs to be checked before this can be merged. Otherwise everything worked perfectly!

After thorough inspection ended up with solution which patches the missing variable name issue. Looked around pages to see if headings look strange but seems to be OK.

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.

This works well now! Great stuff 🦖

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