Skip to content

Add migration to mark images as inactive#952

Open
gunnarvelle wants to merge 4 commits into
masterfrom
image-api-inactive-migration
Open

Add migration to mark images as inactive#952
gunnarvelle wants to merge 4 commits into
masterfrom
image-api-inactive-migration

Conversation

@gunnarvelle
Copy link
Copy Markdown
Member

Fixes NDLANO/Issues#4538

@gunnarvelle gunnarvelle requested a review from a team May 11, 2026 08:25
@gunnarvelle
Copy link
Copy Markdown
Member Author

For prod-dump markerte dette 50000 av 77000 bilder som inaktive.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Kanskje vi skulle gitt denne et navn som gjør at vi ikke er redde for å slette den i fremtiden

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

migration-27-temp-image-ids.txt eller nokke sånt.

val matchesTitleFilter = titlesOpt.exists { titles =>
titles.exists { titleJson =>
titleJson.hcursor.downField("title").as[String].toOption match {
case Some(title) => title.toLowerCase().contains("ikke bruk")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Har vi sjekket om dette matcher på noen bilder vi vil ha egentlig?
Ser for meg "Ikke bruk snus, det er farlig" feks 😅

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Det kan det sjølvsagt treffe på. Kan sjekke litt

Copy link
Copy Markdown
Member Author

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@gunnarvelle gunnarvelle requested review from a team and jnatten May 12, 2026 09:55
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.

2 participants