Project

General

Profile

Bug #83

Description i RSS feed må ta med hele artikkelen

Added by Tom Fredrik Blenning about 8 years ago. Updated almost 8 years ago.

Status:
Resolved
Priority:
High
Start date:
05/25/2018
Due date:
% Done:

0%

Estimated time:

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

nytt-fra-det-offentlige.xml (694 KB) nytt-fra-det-offentlige.xml Bjørnar Hjelmevold, 05/28/2018 09:45 AM

Related issues

Related to Bug #81: Feil i RSS feed authorIn Progress05/25/2018

Actions

History

#1

Updated by Tom Fredrik Blenning about 8 years ago

  • Related to Bug #81: Feil i RSS feed author added
#2

Updated by Bjørnar Hjelmevold about 8 years ago

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?

#3

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å.

#4

Updated by Arve Kirkevik about 8 years ago

Er Bug #83 samme sak som:
«Feature #78
Om å få hele teksten med i artikkelen når den sendes til EFN-listen»?

#5

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.

#6

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.

Also available in: Atom PDF