malmostad / sitevision_assets

Sitevision specific assets used in addition to Global Assets
http://malmostad.github.io/wag-external-v4/
GNU Affero General Public License v3.0
0 stars 1 forks source link

Proxymoduler, anpassa meddelande #349

Closed martent closed 7 years ago

martent commented 7 years ago

Gå igenom proxyportlets och lägg in ett humant felmeddelande istället för det tekniska som kommer om källtjänsten inte går att nå.

@tomasrobertsson ger ett förslag på text som @Svempan godkänner/justerar.

@magnivar håller i aktiviteten.

Aktiviteten ingår som en del i den hälsocheck Bouvet nu gör.

magnivar commented 7 years ago

Alla proxymoduler skall ha den nya texten.

tomasrobertsson commented 7 years ago

Jag tittar på denna senare under veckan.

tomasrobertsson commented 7 years ago

Titt senare i veckan blev idag. Vilka olika fel är det ni vill hantera specifikt? Ska användarna får en generell text eller ska vi ge mer detaljerade fel. För proxyportlet kan man, som ni vet, ställa in ett generella fel eller specificera på olika, 401, 404, 500 osv. Vid granulitet önskas? Jag kan sätta upp möjlighet att skapa olika för alla olika nivåer, generellt meddelande på alla, så kan ni/vi senare sätta upp exakt den text som passar.

martent commented 7 years ago

Tar du denna @magnivar?

magnivar commented 7 years ago

Förslag: (eller vad säger @Svempan) 401 - Du har inte behörighet att besöka sidan. 404 - Sidan du försöker nå hittas inte. 500 - Det är ett tekniskt fel på sidan du vill besöka och kan inte visas. Övriga: Det går inte att visa sidan.

martent commented 7 years ago

För proxy-modulerna handlar det inte om "sidan" utan en mer eller mindre stor ruta på en sida, t.ex. Händer idag. Så meddelandet får bli på temat "informationen kunde tyvärr inte hämtas" eller liknande.

tomasrobertsson commented 7 years ago

Korta koncisa meddelanden som är ganska lika för kunden men som ni/vi som ska felsöka kan säga vad som går fel. Jag implementerar detta på test för externa webben under dagen.

tomasrobertsson commented 7 years ago

Ett test http://www.test.malmo.se/Teststruktur/Proxy-test.html där texten kommer från del på denna sida http://www.test.malmo.se/Redaktorer/Felsida-rss.html Ger 404-fel eftersom adressen i portleten är medvetet fel.

perber14 commented 7 years ago

Jag tycker att texterna borde vara:

401 - Nu blev det fel! Du har inte behörighet att visa informationen. 404 - Nu blev det fel! Det går inte att hitta informationen. 505 - Nu blev det fel! Det går inte att visa informationen. Övriga fel - Nu blev det fel! Det går inte att visa informationen.

Tydlig fel-info och lite mer aktivt språk, som ju är bra för läsförståelsen.

OK, @Svempan?

Svempan commented 7 years ago

OK! @perber14

tomasrobertsson commented 7 years ago

Jag uppdaterar med dessa strängar i mitt testexempel.

tomasrobertsson commented 7 years ago

image det är enkelt att ändra när vi väl implementerat det. Det "jobbiga" är att ställa in alla proxymodeller, peka dem på dessa fält.

tomasrobertsson commented 7 years ago

Publicerade på malmo.se och komin.malmo.se

tomasrobertsson commented 7 years ago

Har hittat en till proxy-portlet som ligger i en mall (för startsidan). Testar lite på test-miljön och fixar även denna.

tomasrobertsson commented 7 years ago

Uppdaterat och jag tog mig rätten att publicera det direkt på produktion. Hoppas ni inte går i taket för det.

Dvs den lösning som finns nu är: 1: För proxyportlets på sidor finns det en sida med felmeddelanden som dessa kan använda. 2: För proxyportlets i mallar finns det en annan MALL med samma felmeddelande som för sidor men enbart användbar för portlets i mallar. Den portlet som använder dessa är den som hämtar bloggar på startsidan..

För båda dessa gäller att ni kan enkelt ändra existerande fel, nya fel läggs till på sidan/mallen samt länkas dit i de portlets som ska hantera dessa fel.

magnivar commented 7 years ago

Klart 2016