Bug #83
Description i RSS feed må ta med hele artikkelen
0%
Description
Grunnet bruken på efn.no er det nødvendig at hele artikkelteksten blir tatt med i RSS feeden.
Optimalt bør det være konfigurerbart hvorvidt bare ingress eller hele artikkelen tas med.
Files
Related issues
History
Updated by Tom Fredrik Blenning about 8 years ago
- Related to Bug #81: Feil i RSS feed author added
Updated by Bjørnar Hjelmevold about 8 years ago
- File nytt-fra-det-offentlige.xml nytt-fra-det-offentlige.xml added
- Status changed from New to Feedback
Se vedlagt XML-fil for eksempel på hvordan resultatet blir. Nytt fra det offentlige blir 800KB stor.
Jeg har byttet fra at alt er i ett <description>-felt til at ingress er i <summary> og brødtekst havner i <content>. Dette er i tråd med eksempelet her: https://en.wikipedia.org/wiki/Atom_(Web_standard)#Example_of_an_Atom_1.0_feed
Imidlertid har jeg vært nødt til å wrappe alt i CDATA, fordi det av og til dukker opp HTML-entities i innholdet som XSLT-transformasjonen ikke setter særlig pris på. Det er mulig det finnes en løsning på det hvis jeg parser teksten, men det vil være en litt tidkrevende jobb med potensielle edge cases som kan dukke opp og skape krøll.
Når det gjelder mulighet til å konfigurere hvorvidt bare ingress eller også brødteksten skal med, så er dette for tiden konfigurerbart på hver RSS-feed. Er det ønskelig at dette også skal være konfigurerbart på hver artikkel?
Updated by Tom Fredrik Blenning about 8 years ago
- Status changed from Feedback to In Progress
Testet det og oppdaget at det er flere feil i feeden ihht. standard.
Uansett det viser seg at det er underfeltet name som må være tilstede. Manipulerte filen på https://div.efn.no/test/rss/nytt-fra-det-offentlige.rss da parses den korrekt av rss2email og det er det som er pri akkurat nå.
Updated by Arve Kirkevik about 8 years ago
Updated by Tom Fredrik Blenning about 8 years ago
Dette dreier seg om å legge hels teksten ut i RSS feeden, men #83 dreier seg om hva man skal sende ut på EFN-listen. Remaileren sender ut det som legges inn i RSS-feeden.
Updated by Bjørnar Hjelmevold almost 8 years ago
- Status changed from In Progress to Resolved
Jeg tror denne er løst, men den er ikke grundig testet ennå. Forrige test ble mislykket/avbrutt på grunn av alle e-postene som sto i kø.
name skal nå være med i alle items i feeden, det brukes en fallbackverdi hvis name mangler. Jeg mener dette er noe som vi fikk bekreftet under vår forrige test for 2 måneder siden.
Jeg fant en bug med at ikke-eksterne URL-er var relative og uten efn.no i forkant. Jeg har løst det raskt ved å erstatte alle forekomster av href="/ i brødteksten med href="https://efn.no/ i kontrolleren før visning. Hvis denne løsningen er for lite fleksibel så kan det gjøres dynamisk ut fra miljø, men vil i så fall kreve litt mer arbeid fra min side fordi jeg må dra inn request-objektet.
Sånn som det er nå, er det konfigurerbart hvordan brødteksten kommer inn i feeden.
Enten ingress og brødtekst separat:
<summary>Ingress som tekststreng strippet for eventuelle HTML-tags</summary>
<content><p>Brødtekst som HTML<p></content>
Eller ingress og brødtekst samlet:
<summary><p>Ingress og brødtekst som HTML</p></summary>
<content><p>Kopi av ingress og brødtekst som HTML<p></content>
Innstillingen for dette ligger som en checkbox "Merge summary and content body" på RSS-feed-innholdet inne i Content Studio. Default innstilling at ingress og brødtekst holdes atskilt.
Ny versjon av RSS-appen er nå deployet i prod.