Project

General

Profile

Enhancement #3302

Arrangement kan forgå på forskellige lokaliteter i samarbejde med eksterne arrangører

Added by Marianne Sivertsen about 2 years ago. Updated over 1 year ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Arrangementer, Driftsvedligehold - Oversættelser (udarbejdes før udrulning)

Description

Jeg opretter en event.

Jeg skal vælge et tilknytningsbibliotek, selv om arrangementet foregår et helt andet sted.

Inde på eventen figurerer tilknytningsbiblioteket, så det kan give misforståelser af, hvor det foregår.

Og hvis jeg går ind på siden Arrangementer og søger på biblioteksnavnet, dukker dette arrangement op.

Kunne man ikke forestille sig, at man havde en ekstra post, der hed 'Andre steder'?

Lokalitet.docx (479 KB) Lokalitet.docx Marianne Sivertsen, 02/18/2018 12:05 PM
Skærmbillede 2018-10-02 15.35.37.png (144 KB) Skærmbillede 2018-10-02 15.35.37.png stylingfejl? Gitte Barlach, 10/02/2018 03:46 PM
Skærmbillede 2018-10-02 16.23.57.png (242 KB) Skærmbillede 2018-10-02 16.23.57.png https://upgrade-fbs.ddbcms.dk/services/event_list Gitte Barlach, 10/02/2018 04:26 PM
Skærmbillede 2018-10-02 16.07.15.png (214 KB) Skærmbillede 2018-10-02 16.07.15.png https://upgrade-fbs.ddbcms.dk/ding-redia-rss/event Gitte Barlach, 10/02/2018 04:27 PM
Skærmbillede 2018-10-04 09.07.22.png (50.5 KB) Skærmbillede 2018-10-04 09.07.22.png Place/Room skal IKKE vises til slutbrugeren når lokalitet er udfyldt Gitte Barlach, 10/04/2018 09:17 AM
Skærmbillede 2018-10-04 09.16.25.png (17.5 KB) Skærmbillede 2018-10-04 09.16.25.png Place/Room skal IKKE vises til slutbrugeren når lokalitet er udfyldt Gitte Barlach, 10/04/2018 09:17 AM
Skærmbillede 2018-10-15 10.26.08.png (19.3 KB) Skærmbillede 2018-10-15 10.26.08.png - et arrangement kan være arrangeret af biblioteket, men rent fysisk kan foregå på en anden lokalitet end på biblioteket Gitte Barlach, 10/15/2018 10:29 AM
Skærmbillede 2018-10-15 10.26.28.png (73.9 KB) Skærmbillede 2018-10-15 10.26.28.png - et arrangement kan være arrangeret af biblioteket, men rent fysisk kan foregå på en anden lokalitet end på biblioteket Gitte Barlach, 10/15/2018 10:29 AM
Skærmbillede 2018-10-15 10.53.52.png (324 KB) Skærmbillede 2018-10-15 10.53.52.png - et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, og rent fysisk foregår det på x-lokalitet i biblioteket Gitte Barlach, 10/15/2018 10:55 AM
Skærmbillede 2018-10-15 10.53.11.png (99.8 KB) Skærmbillede 2018-10-15 10.53.11.png - et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, og rent fysisk foregår det på x-lokalitet i biblioteket Gitte Barlach, 10/15/2018 10:55 AM
Skærmbillede 2018-10-15 11.06.38.png (400 KB) Skærmbillede 2018-10-15 11.06.38.png - et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, men rent fysisk kan det foregå på en anden lokalitet end på biblioteket Gitte Barlach, 10/15/2018 11:09 AM
Skærmbillede 2018-10-15 11.07.19.png (152 KB) Skærmbillede 2018-10-15 11.07.19.png - et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, men rent fysisk kan det foregå på en anden lokalitet end på biblioteket Gitte Barlach, 10/15/2018 11:09 AM
Skærmbillede 2018-10-15 11.11.09.png (24.4 KB) Skærmbillede 2018-10-15 11.11.09.png Hvordan skal jeg kunne vælge måned, når dropdown-menu´en dækker over dele af kalenderinterfacet?? Gitte Barlach, 10/15/2018 11:13 AM

Related issues

Related to DDB CMS - Bug #3180: Visning af arrangement med alternativ afholdelsesadresseClosed
Related to DDB CMS - Enhancement #3303: Arrangement tilknyttet flere biblioteker optræder én gang for hvert sted på forsidenNeed more info
Related to DDB CMS - Enhancement #3224: Arrangementer. Nyt arrangør-felt på et arrangement TESTES MED PR I 3302Resolved
Related to DDB CMS - Bug #1757: Listvisning af arrangementer der ikke er tildelt et bibliotekReady for development
Related to DDB CMS - Enhancement #3874: Listen med arrangementer under event-organizer skal vises som andre arrangementslisterReady for development
Has duplicate DDB CMS - Bug #2864: Tilknyttet bibliotek vises, når en begivenhed foregår på alternativ adresseClosed

History

#1 Updated by Rolf Madsen about 2 years ago

  • Description updated (diff)
  • Status changed from New to Need more info
  • Assignee set to Marianne Sivertsen
  • Target version set to DDB CMS - Analyse og prioritering udestår

Så du vil gerne have fjernet "Bibliotek: Hårlev Bibliotek" hvis der er angivet en alternativ lokalitet?

#2 Updated by Marianne Sivertsen about 2 years ago

Ja.

Men i virkeligheden handler det måske bare om, at jeg skal have mulighed for ikke at vælge et tilknytningsbibliotek ved oprettelsen.

#3 Updated by Rolf Madsen about 2 years ago

Et arrangement skal tilknyttes et bibliotek for at vi kan placeret det i hjemmesidens link struktur.

Ellers ville det "svæve" uden tilhørsforhold.

#4 Updated by Marianne Sivertsen about 2 years ago

Ok.

Og hvis man ikke kan løse den ved så at have en 'tilknytningsfilial' der hedder noget á la 'Andet sted' - så må det handle om (som du skriver) ikke at få vist 'Hårlev Bibliotek', når det foregår et andet sted.

#5 Updated by Rolf Madsen about 2 years ago

  • Subject changed from Events - Valg af tilknytningsbibliotek medfører risiko for misforståelser to Arrangementer der ikke afholdes på et bibliotek skal ikke have vist det tilknyttede biblioteksnavn

#6 Updated by Rolf Madsen about 2 years ago

  • Status changed from Need more info to Needs prioritization
  • Assignee changed from Marianne Sivertsen to Christel Krabbenhøft
  • Target version changed from DDB CMS - Analyse og prioritering udestår to Release 29-2 - Bugfixes (B14)

#7 Updated by Rolf Madsen about 2 years ago

  • Status changed from Needs prioritization to Needs design decision
  • Assignee changed from Christel Krabbenhøft to Gitte Barlach

Vi må lige have en runde mere på den her ...

Det viser sig at der er nogle biblioteker der både bruger bibliotek men også lokalitet til at indikere hvor på biblioteket arrangementet finder sted.

#8 Updated by Rolf Madsen about 2 years ago

  • Related to Bug #3310: Nodequeue-liste slår ikke igennem på oversigten over biblioteker added

#9 Updated by Rolf Madsen about 2 years ago

  • Has duplicate Bug #2864: Tilknyttet bibliotek vises, når en begivenhed foregår på alternativ adresse added

#10 Updated by Marianne Sivertsen about 2 years ago

Men så ville det jo give ekstrameget mening at have et dummy-bibliotek á la 'andet sted' til tilfælde som mit - så ville det stadig være muligt med kombi-modellen, som du nævner at nogle biblioteker har brug for.

#11 Updated by Rolf Madsen about 2 years ago

@Gitte, vil du ikke byde ind her, jeg ved I havde nogle tanker omkring håndteringen af problemstillingen.

#12 Updated by Gitte Barlach about 2 years ago

  • Assignee changed from Gitte Barlach to Rolf Madsen

Yes, det prøver jeg her :

Vi foreslår en løsning der både i frontend og backend kan tage højde for at:

- et arrangement kan være arrangeret af biblioteket, og rent fysisk foregår det på x-lokalitet i biblioteket
- et arrangement kan være arrangeret af biblioteket, men rent fysisk kan foregå på en anden lokalitet end på biblioteket
- et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, og rent fysisk foregår det på x-lokalitet i biblioteket
- et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, men rent fysisk kan det foregå på en anden lokalitet end på biblioteket

Eksempler:

Lokalitet: Hele biblioteket
Bibliotek: Beder-Malling Bibliotek

Lokalitet: Skolens kantine
Bibliotek: Egå Kombi-bibliotek

Lokalitet: Udenfor biblioteket
Bibliotek: Hasle Bibliotek

Lokalitet: LitteraturStedet
Bibliotek: Åby Bibliotek

Lokalitet: I børnelitteraturen
Bibliotek: Gellerup Bibliotek

Lokalitet: P-pladsen
Bibliotek: Lystrup Bibliotek

Lokalitet: Haven
Bibliotek: Risskov Bibliotek

Lokalitet: 1. etage
Bibliotek: Højbjerg Bibliotek

Lokalitet: Salen
Bibliotek: Viby Bibliotek

Lokalitet: Bassinet 
Bibliotek: Bavnehøj Bibliotek

Lokalitet: Dokk1, Store Sal
Bibliotek: Hovedbiblioteket

Lokalitet: Dokk1, Niveau 2.1, Pusterummet
Bibliotek: Hovedbiblioteket

Eksempel hvor "lokaliteten har anden adresse" er udfyldt:

Lokalitet: Tilst-Kasted Sognegård, Havkærvej 9, 8381 Tilst
Arrangør: Tilst Bibliotek

Forslaget ændrer ikke ved sorteringsfunktionen "vælg bibliotek" på siden /arrangementer
Forslaget ændrer i backend og frontend - se skitserne her:
Visning af arrangement i oversigten /arrangementer: https://projects.invisionapp.com/share/TJFZOIOZXFN#/screens/282557673_03-01_Oversigt-Det-Sker
Backend funktionalitet samt visning af selve arrangementet: https://projects.invisionapp.com/share/TJFZOIOZXFN#/screens/280893393_01-02_Admin

 

#13 Updated by Gitte Barlach about 2 years ago

  • Status changed from Needs design decision to Ready for development
  • Assignee changed from Rolf Madsen to Christel Krabbenhøft

#14 Updated by Christel Krabbenhøft about 2 years ago

  • Related to Bug #3180: Visning af arrangement med alternativ afholdelsesadresse added

#15 Updated by Christel Krabbenhøft about 2 years ago

  • Related to deleted (Bug #3310: Nodequeue-liste slår ikke igennem på oversigten over biblioteker)

#16 Updated by Christel Krabbenhøft about 2 years ago

  • Related to Bug #3180: Visning af arrangement med alternativ afholdelsesadresse added

#17 Updated by Rolf Madsen about 2 years ago

  • Related to Enhancement #3303: Arrangement tilknyttet flere biblioteker optræder én gang for hvert sted på forsiden added

#18 Updated by Christel Krabbenhøft almost 2 years ago

  • Assignee changed from Christel Krabbenhøft to Rasmus Høymann Laursen

#19 Updated by Rasmus Høymann Laursen almost 2 years ago

  • Assignee changed from Rasmus Høymann Laursen to Steen Holten-Andersen

#20 Updated by Steen Holten-Andersen almost 2 years ago

  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

Issue https://platform.dandigbib.org/issues/3466 bliver nødt til at blive merged i core før vi kan komme videre, da alle felt-grupper er forsvundet i backend (se issue 3466). 

Derfor bliver alt markup ødelagt på event, og det bliver mere end dobbelt arbejde at lave, hvis ikke ovenstående bliver løst først.

#21 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

Hej Steen 

Har du tid at kigge på denne, nu da #3466 er løst?

#22 Updated by Steen Holten-Andersen almost 2 years ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

PR: https://github.com/ding2/ding2/pull/1119

I forhold til de optegende skitser har jeg følgende kommentarer:

Der er oprettet et 'place / Room' særskilt felt (required). Det er ikke umiddelbart muligt at merge et ekstra felt ind i 'location' feltet. Har i den forbindelse undladt det ønskede check-felt 'lokaliteten har en anden adresse'. Hvis der er indtastet værdier i det eksisterende 'location' felt, opfører felterne som angivet hvis feltet 'lokaliteten har en anden adresse' (da der jo netop indtastes en alternativ adresse). 

Har yderligere en lille notits til dette PR: 

Som ønsket bliver det nye (required) felt 'place / room' printet ud i teaser-visning istedet for som hidtil 'location'. Dette har dog nogle bagudkompabilitets-problemer.

Alle events som ikke har udfyldt det nye 'place / room' felt, vil herved ikke få vist nogen adresse i teaservisning, før events'ne er blevet opdateret til at bruge det nye felt.

Eftersom ønsket er at det nye 'place / room' felt er required, kan der yderligere opstå en lidt underlig use-case :

Arrangementet afholdes på en anden adresse (location feltet er udfyldt), og data indtastet i 'location' feltet er fyldestgørende for at beskrive arrangementet. Nu skal man så yderligere skrive noget i 'place / room' - selvom det ikke er nødvendigt.

#23 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs
  • Priority changed from Normal to High

#24 Updated by Kasper Garnæs over 1 year ago

  • Status changed from Needs code review to Needs design decision
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Reviewed. Jeg har en række kommentarer til hvordan dette er løst.

Generelt så overlapper dette issue med #3224 hvor der også bliver tilføjet en Organizer til events. Her bliver det modelleret som et link. I #3224 bliver der oprettet en ny taksonomi til formålet. En ny taksonomi gør det lettere at "genbruge" organizers på tværs af arrangementer og knytte yderligere data til. Til gengæld kan arbejdsgangen være lidt mere kompleks idet en Organizer skal oprettes før den kan tilknyttes et arrangement.

Forretningen må beslutte hvilken løsning der foretrækkes.

#25 Updated by Rolf Madsen over 1 year ago

@Gitte, jeg foretrækker umiddelbart en ny taksonomi. Hvis jeg har forstået det korrekt vil det svare til at man opretter en taksonomi før man kan vælge en af de oprettede værdier fra taksonomien under oprettelse af et arrangement. Det er en arbejdsgang bibliotekerne i forvejen er bekendte med.

#26 Updated by Rolf Madsen over 1 year ago

  • Related to Enhancement #3224: Arrangementer. Nyt arrangør-felt på et arrangement TESTES MED PR I 3302 added

#27 Updated by Gitte Barlach over 1 year ago

  • Status changed from Needs design decision to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

Jeg er enig i at det bedste vil være at der bliver oprettet en ny taksonomi. Det sikrer konsistent sprogbrug, hvilket f.eks. kan være en fordel hvis man på et senere tidspunkt ønsker at kunne filtrere samt søge på arrangør. 

@Steen: vil du se på sagen igen ud fra den forudsætning at det ønskede arrangør-felt løses med #3224 ?

#28 Updated by Rolf Madsen over 1 year ago

  • Related to deleted (Bug #3180: Visning af arrangement med alternativ afholdelsesadresse)

#29 Updated by Rolf Madsen over 1 year ago

  • Related to Bug #1757: Listvisning af arrangementer der ikke er tildelt et bibliotek added

#30 Updated by Steen Holten-Andersen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

#31 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#32 Updated by Gitte Barlach over 1 year ago

  • Subject changed from Arrangementer der ikke afholdes på et bibliotek skal ikke have vist det tilknyttede biblioteksnavn to Arrangement kan forgå på forskellige lokaliteter i samarbejde med eksterne arrangører

Set i forhold til løsningen havde dette issue et noget misvisende titel; har derfor opdateret titlen. 

#33 Updated by Kasper Garnæs over 1 year ago

  • Status changed from Needs code review to Reviewed

Godkendt og merged.

#34 Updated by Kasper Garnæs over 1 year ago

  • Status changed from Reviewed to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Rettelsen er en del af i 4.5.0-rc4.

#35 Updated by Gitte Barlach over 1 year ago

Når denne testes skal vi huske også at teste:
-  feedet til App´en samt selve app´en
-  feedet /services/event_list jvf. #38
samt gøre opmærksom på ændringerne ift. leverandører der har skærmprodukter, der er afhængige af contentservicen

 

#36 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Stefan Søndervang

#37 Updated by Gitte Barlach over 1 year ago

testet på upgrade-fbs med 4.5.0-rc4

a1) Frontend: Såfremt "Lokalitet" er udfyldt (dvs arrangementet foregår et ANDET STED end på biblioteket), skal "Place/room" ikke vises noget steds i grænsefladen til slutbrugeren 
a2) Backend: Såfremt "Lokalitet" er udfyldt (dvs arrangementet foregår et ANDET STED end på biblioteket), bør "Place/room" ikke være påkrævet
a3) Feltet "Fulde navn" under LOKALITET er præudfyldt med teksten "Location"; dette ønskes fjernet. I stedet bør der laves en bedre hjælpetekst 

b) Arrangør skal kun vises inde på selve arrangementet
c) en stylingfejl må det være? se vedlagte skærmdump "stylingfejl?"
d) jeg undrer mig over at den engelske oversættelse: inde på arrangementet hedder det "Organizer", men på forsiden og i oversigten / hedder det "arranged by". Kunne det ikke bare hedde det samme?
e) integration til Biblioteket app´en
e1) vi har ikke fået specificeret at rettelserne, herunder det nye Orgnizer-felt, skal eksponeres i /ding-redia-rss/event
Feedet anser bare pågældende bibliotek som sted/location:

<content-rss:arrangement-location>Hovedbiblioteket</content-rss:arrangement-location>

e2) App-po skal afklare hvordan visning i app´en skal se ud, samt aftale udvikling af dette

f) ift. /services/event_list kan jeg ikke verificere at Organizer er med. 
 

#38 Updated by Rolf Madsen over 1 year ago

  • Related to Enhancement #3874: Listen med arrangementer under event-organizer skal vises som andre arrangementslister added

#41 Updated by Gitte Barlach over 1 year ago

  • Status changed from Technical test to Reviewed - Needs info/rework

#42 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Stefan Søndervang to Rasmus Høymann Laursen

#43 Updated by Gitte Barlach over 1 year ago

  • Priority changed from High to Urgent

#44 Updated by Rolf Madsen over 1 year ago

  • Kategorier Driftsvedligehold - Oversættelser (udarbejdes før udrulning) added

#45 Updated by Rasmus Høymann Laursen over 1 year ago

  • Assignee changed from Rasmus Høymann Laursen to Steen Holten-Andersen

#46 Updated by Rolf Madsen over 1 year ago

msgid "Add one or more organizer(s) to your event."
msstr ""

msgid "Event organizer link"
msstr ""

msgid "Place / Room"
msstr ""

msgid "Use this to direct your guests to the correct place"
msstr ""

msgid "arranged by"
msstr ""

msgid "Organizers"
msstr ""

msgid "Organizer"
msstr ""

#47 Updated by Steen Holten-Andersen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

Hej Gitte. Jeg er meget i tvivl om dine kommentarer. Løsningen er udarbejdet på baggrund af skitserne her: invision skitser vedlagt i #12

a1) : I skitserne fremgår det i teaservisning under 'Med anden adresse og arrangør' - at Sted/rum vises, men ikke resten? Også på den fulde visning vises sted/rum?

a2) : Gjorde allerede i #22 opmærksom udfordringer ved et required place/room felt som der var ønsket, men dette kan vi sagtens ordne (kræver bare lidt klarhed over den endelige løsning).

b) I invision udgaven vises arrangør også i oversigten? 

d) Aranged by er taget direkte fra Invision skitserne, hvor det hedder: arrangeret af. 

Så - i forhold til ovenstående, vil det være rart med en afklaring af præcis hvad der ønskes i alle scenarier, hvis løsningen ikke er angivet i de leverede skitser - således at vi alle er sikre på at Issue'et løses korrekt. 

Mht. e/e1/e2/f - ville det måske være en ide at udskille dette i et seperat Issue - for ikke at blande tingene sammen - således at dette Issue klarer håndteringen af løsningen på Web, mens der oprettes et særskilt Issue til at håndtere ændringen i app?

 

#48 Updated by Gitte Barlach over 1 year ago

Tak for feedback, Steen. Vi har på den baggrund givet den en runde mere:


a1) vi holder fast i at der skal være mulighed for at angive place /room for andre adresser også. Argumentet herfor er, at der vil være mange som også har flere forskellige lokaler, som kan være i spil, så derfor vil det være en god hjælp til gæsten for at finde vej.  

a2) Til gengæld skal det så være frivilligt om man vil udfylde place /room. Argumentet herfor er, at der er steder (såvel biblioteker som eksterne lokationer), der er så små, at det ikke giver mening at feltet er påkrævet. (så du havde ret her)

a3) visning: 
selve arrangementet: Hvis feltet place / room ikke udfyldt skal det ikke vises. Hvis det er udfyldt skal det vises umiddelbart før adressen på den eksterne lokation.  
teaservisning: vi holder fast i den nuværende visning: place /room vises (hvis det er udfyldt), efterfulgt af værdien af feltet "Fulde navn" under Sted i backend

b) ja det er rigtigt; Vi er dog blevet klogere og kan se at arrangør hurtigt kommer til at fylde for meget her. 

d) ja det er rigtigt. Beklager. Jeg tænker at det ville være fint at få en oversættelse mindre, især set i lyset af b)
 

ift. e/e1/e2/f. God pointe. Jeg opretter en ny sag. 

#49 Updated by Gitte Barlach over 1 year ago

  • Status changed from Need more info to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

#50 Updated by Steen Holten-Andersen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

He Gitte - jeg forstår det således:

a1 / a2 :

Place/room bliver ikke required, er det korrekt?

Teaser visning:

Hvis Place/Room er udfyldt alene (ingen anden lokalitet) - vises: [Place/room] - [Biblioteksnavn], er det korrekt?

Hvis Place/Room er udfyldt med anden lokalitet - vises: [Place/Room] - [Lokalitetsnavn], er det korrekt?

 

b / c :Arrangør fjernes fra teaser-visninger, og derfor bliver strengen 'Arranged by' overflødig og udgår, er det korrekt?

- Hvis vi er enige, går jeg igang med det samme :-)

#51 Updated by Gitte Barlach over 1 year ago

ja, ovenstående er korrekt opsummeret!

#52 Updated by Rasmus Høymann Laursen over 1 year ago

  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

#53 Updated by Gitte Barlach over 1 year ago

Når vi nu er ved det: 

kan vi undgå at der står "Danmark" ? det burde jo være indlysende for langt de fleste adressers vedkommende at det er i Danmark. 
dvss. kan man i dropdon-menu´en under "land" få lov at vælge ingen (dvs. en tom værdi), og hvis dette er valgt skal der ikke vises noget til slutbrugerne.
Eller er der en anden mulighed for at man kan lade feltet stå tomt?

#54 Updated by Gitte Barlach over 1 year ago

  • Status changed from Need more info to Reviewed - Needs info/rework

#55 Updated by Steen Holten-Andersen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

PR: https://github.com/ding2/ding2/pull/1211

Har fjernet muligheden for at vælge land. Kunne ikke genskabe autoudfyldt Location. Er du sikker på at det ikke er browseren der udfylder det, og ikke systemet?

#56 Updated by Gitte Barlach over 1 year ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Steen Holten-Andersen

vedr. "Har fjernet muligheden for at vælge land."

ah .. helt at fjerne muligheden for at vælge land er ikke godt af hensyn til Dansk Centralbibliotek for Sydslesvig. 
Det var derfor jeg foreslog "kan man i dropdon-menu´en under "land" få lov at vælge ingen (dvs. en tom værdi), og hvis dette er valgt skal der ikke vises noget til slutbrugerne.
Eller er der en anden mulighed for at man kan lade feltet stå tomt?"

#57 Updated by Steen Holten-Andersen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Steen Holten-Andersen to Gitte Barlach

Ok - men som det var, kunne man alligevel kun vælge Danmark. Der er ingen ud af boksen løsning på ovennævnte, felttypen har to muligheder:

Afgræns lande der kan vælges.

Hvis kun ét land er valgt som mulighed (som var tilfældet før dette commit) - fjern da feltet. Ellers skal vi hooke os ind i felttypens indstillinger, som gør kodebasen mere sårbar overfor opdateringer.

#58 Updated by Gitte Barlach over 1 year ago

okay - så vil jeg foreslå at fjerne land, som dit pr gør.  (troede bare at man på en aller anden måde kunne indsætte en ydelrigere værdi, da det lignede en dropdown boks)
 

#59 Updated by Gitte Barlach over 1 year ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Gitte Barlach to Kasper Garnæs

#60 Updated by Kasper Garnæs over 1 year ago

  • Status changed from Needs code review to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Godkendt og merged.

#61 Updated by Rolf Madsen over 1 year ago

Nu har jeg givet et bud på oversættelserne:

msgid "Add one or more organizer(s) to your event."
msstr "Tilføj en eller flere arrangører til dit arrangement."

msgid "Event organizer link"
msstr "Link til arrangementets arrangør"

msgid "Place / Room"
msstr "Lokalitet"

msgid "Use this to direct your guests to the correct place"
msstr "Benyttes til at dirigere dine gæster til det korrekte lokale"

msgid "arranged by"
msstr "Arrangeret af"

msgid "Organizers"
msstr "Arrangører"

msgid "Organizer"
msstr "Arrangører"

#62 Updated by Gitte Barlach over 1 year ago

Jeg tester denne. 

#63 Updated by Gitte Barlach over 1 year ago

Har testet flg. usecases:

- et arrangement kan være arrangeret af biblioteket, og rent fysisk foregår det på x-lokalitet i biblioteket
Teaservisning: ok
Visning af selve arrangementet: ok. 

- et arrangement kan være arrangeret af biblioteket, men rent fysisk kan foregå på en anden lokalitet end på biblioteket
Teaservisning: det burde være tydeligere at biblioteket er arrangør. Så der bør enten stå "Arrangør" foran navnet på biblioteket eller også skal vi slet ikke vise navnet på biblioteket i teaservisningen. Jeg hælder til det sidste for at gøre det enkelt for brugeren. Beklager hvis vi ikke har være tydeligere nok omkring dette.
Visning af selve arrangementet: ok. 

- et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, og rent fysisk foregår det på x-lokalitet i biblioteket
Teaservisning:
Visning af selve arrangementet:


- et arrangement kan være arrangeret af biblioteket i samarbejde med x-arrangør, men rent fysisk kan det foregå på en anden lokalitet end på biblioteket
Teaservisning: ok
Visning af selve arrangementet: ok. 

Rettet: dette skyldes måske at browseren prøver at autoudfylde.... anyhow: det sker ikek altid, og man kan omgå det ved at klikke et sted på den del af kalenderen der trods alt er synlig; så jeg frafalder...
Af en eller anden grund er der sket en ændring af UX i backend:
Når man skal inddatere en dato og vil vælge fra kalenderen, glider der en dropdown ned med den dato, der allerede står i inddateringsfeltet. Og den dækker over dato-vælgeren, så man reelt ikek kan bruge kalender-interfacet til at vælge måned. Det er rimeligt irriterende, og det bør rettes Det har ikke tidligere været sådan. 
 

#64 Updated by Gitte Barlach over 1 year ago

  • Status changed from Technical test to Resolved (tag version)

Also available in: Atom PDF