Skip to content

UHF-X: migration updates#914

Draft
hyrsky wants to merge 5 commits intodevfrom
UHF-X-migration-updates
Draft

UHF-X: migration updates#914
hyrsky wants to merge 5 commits intodevfrom
UHF-X-migration-updates

Conversation

@hyrsky
Copy link
Copy Markdown
Contributor

@hyrsky hyrsky commented Feb 27, 2026

UHF-0000

What was done

  • Clean deprecated code.
  • Run migrations without --update flag
  • Clean migration maps when job listings are deleted.

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
  • Switch to feature branch
    • git fetch && git checkout UHF-X-migration-updates,
  • Run code updates
    • composer install
    • make drush-deploy drush-locale-update drush-cr
  • This change requires re-importing all job listings:
    • drush migrate:reset-status helfi_rekry_jobs && drush migrate:rollback helfi_rekry_jobs && drush sqlq "DROP TABLE migrate_map_helfi_rekry_jobs"

How to test

  •  drush migrate:import helfi_rekry_jobs
  • drush migrate:import helfi_rekry_jobs again, should see Processed 0 items (0 created, 0 updated, 0 failed, 0 ignored).
  • drush scheduler:cron
  • drush sapi-c job_listings; drush sapi-rt job_listings; drush sapi-i job_listings
  • Job listing search should work as expected.
  • Check that the code follows our standards

Links to related PRs

Removing

  protected $strictConfigSchema = FALSE; // phpcs:ignore

caused a lot of errors, because the test was installing config/install.
I removed a bunch of settings from the install config so that the test
passes. I don't think there is a point in duplicating most of this
config with conf/cmi folder.
@hyrsky hyrsky force-pushed the UHF-X-migration-updates branch from 307135e to f1626db Compare February 27, 2026 13:57
@hyrsky hyrsky changed the title Uhf x migration updates UHF-X: migration updates Feb 27, 2026
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@hyrsky hyrsky marked this pull request as draft March 10, 2026 13:19
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.

1 participant