Converts norgesfilm-urls for new format #612
Conversation
| .forEach(embed => { | ||
| val url = embed.attr("data-url") | ||
| if (url.contains("ndla.filmiundervisning.no/film/ndlafilm.aspx?")) { | ||
| embed.attr("data-url", url.replace("/ndlafilm.aspx?", "/")): Unit |
There was a problem hiding this comment.
Skal de nye url'ene være typ: https://ndla.filmiundervisning.no/film/filmId=13074?
Veldig rart å ha = uten at det er query parameter med 🤷
There was a problem hiding this comment.
Sant det. Har ikkje tenkt på.
Vil dere ha mulighet til å programmere script endring av iframe URLer dere har i dag på et avtalt tidspunkt?
(Vi kommer til å gjøre klar ny URL struktur som er tilgjengelig samtidig med den gamle.)
URL endres fra: https://ndla.filmiundervisning.no/film/ndlafilm.aspx?filmId=400199
Til feks: https://ndla.filmiundervisning.no/film/400199
There was a problem hiding this comment.
Oki!
Det er jo ikke sånn at det gjør noe, bare litt merkelig 😄
There was a problem hiding this comment.
Avventer uansett beskjed fra norgesfilm før vi tar dette inn.
|
Nye urler er på formatet /film/id. Stagingurler er https://ndla-staging.azurewebsites.net/ndla/11970 Har hacka til en test på https://ndla-frontend-pr-2535.vercel.app/r/ndla-film/catfish/539f0a60b6 |
gunnarvelle
force-pushed
the
convert-norgesfilm-urls
branch
from
bd1406c to
86a171c
Compare
jnatten
left a comment
jnatten
left a comment
There was a problem hiding this comment.
Stilig med ndla2 i navnet på url'en 🥲
Hadde du funne på nokke sånt, hadde eg ikkje latt det passere... |
Ny url ikkje tilgjengelig endå, så må vente med denne til vi får beskjed om det.