search

alternativ-nu / retrosearch

R-paket med sökfunktionalitet för artiklar publicerade i Tidningen Åter
https://retrosearch.recraft.me
GNU Affero General Public License v3.0
0 stars 0 forks source link

Köpknappen lägger till i varukorgen men öppnar inte separat fönster #6

Open recraft-ou opened 5 years ago

recraft-ou commented 5 years ago

Någon ändring i HTML-koden för köp-knappen behövs så att fönstret öppnas pss som när man klickar på bilden?

TidningenAter commented 5 years ago

För mig lägger köpknappen inget i varukorgen.

recraft-ou commented 5 years ago

Jag får att det läggs till i varukorgen, men jag ser det först när jag klickar på thumbnailen och går till butiken. Vilket är förvirrande. Ser du samma sak? Detta ska åtgärdas.

TidningenAter commented 5 years ago

Ja, det funkar så i Chrome på datorn men inte i Safari eller på mobilen.

recraft-ou commented 5 years ago

Tack för denna info, jag ska gräva i detta och återkommer.

recraft-ou commented 4 years ago

Jag har försökt åtgärda detta nu på ett sätt som jag tycker verkar funka i Chromium och Firefox, men har inte testat andra klienter ännu.

TidningenAter commented 4 years ago

Jag testade i Chrome där det funkar utmärkt. Smidigt att man kom till kassa sidan direkt också. Då ser man tidningen man lagt i korgen även om högerspalten skulle råka vara dold pga smal skärm.

Tyvärr funkar det inte i Safari.

TidningenAter commented 4 years ago

För mig funkar det fortfarande bra att lägga saker i varukorgen med Chrome, men man hamnar därefter på https://retrosearch.recraft.me/ Bäst är om man stannar på www.alternativ.nu/artiklar, och inte heller kommer till butiken alls, eftersom du nu lagt till en så bra "Till kassan"-knapp

recraft-ou commented 4 years ago

Detta är lite kopplat till #20 det där med Safari, tror jag. Detta har kanske att göra med att sidan ligger i en iframe, tror jag. Kanske skulle sidan öppnas med target="_top" men jag använder "_blank". Lite osäker på vad som är den minst dåliga lösningen här. Pga olika domäner är inblandade kanske XHR spelar in också (tror jag efter att ha kollat på http://softwareas.com/cross-domain-communication-with-iframes/). Jag hade velat slippa göra något jquery js för en sån enkel sak, det känns som att det borde vara onödigt...

recraft-ou commented 4 years ago

Klagar Safari på något? Vore bra med ett felmeddelande, kanske från Safari's console? Denna är kanske gammal men man ska kunna öppna någon slags "Develop Mode" och se eventuella felmeddelanden genom att göra något i stil med detta: https://apple.stackexchange.com/questions/58948/how-do-i-open-the-javascript-console-in-safari-6

TidningenAter commented 4 years ago

Men ska vi försöka att inte ha det i en iframe då? Det kanske ändå är något som strular till saker t ex om man vill få till länkar för att förvälja en viss filtrering eller p-nummer.

TidningenAter commented 4 years ago

Jag ser inget konstigt i konsollen, förutsom att det saknas en massa css-filer. Men jag tror jag kommit på att safari nobbar begäran om att lägga i varukorgen när man bara använder den rena länken https://www.alternativ.nu/butik/cart?add=1&id_product=259 utan tillägget "&token=..." http://www.alternativ.nu/butik/cart?add=1&id_product=261&token=c1359edf2c70acccc6a966b718058cf0 ...Som blir när man kommer inifrån butiken. För chrome funkar det både med och utan. Kanske någon säkerhetsinställning i min safari-app?

recraft-ou commented 4 years ago

Det där token-värdet har jag försökt jaga rätt på ifall man skulle kunna få det från Prestashop-API:et, men jag har inte lyckats hitta det trots ganska idoga sökningar. Om man kunde få tag på den på något bra sätt kan vi testa det. Jag tror att jag labbade lite med detta för länge sedan men kommer inte ihåg längre var det landade. I värsta fall skulle man behöva hämta den från klientsidan, men det är kanske inte är så. Isånafall kan servern få tag på värdet genom att scrapea butikssidan om nu denna token inte är personbunden.

Jag har följande länk till senaste numret: http://www.alternativ.nu/butik/cart?add=1&id_product=257&token=c1359edf2c70acccc6a966b718058cf0

Flytta till en subdomän på alternativ.nu är ett alternativ som är värt att utforska i samma veva. Serverns FQDN kunde då ändras till exv artiklar.alternativ.nu (då är det iaf samma domän även om subdomänen skiljer). Därmed behöver man inte ändra i den befintliga miljön för alternativ.nu alltför mycket exv genom att introducera docker eller sätta upp en proxy för att mappa om trafiken till en ny tjänst.

För att prova det alternativet så får jag be dig om att göra en DNS inställning på Loopia för alternativ.nu, lägga till en subdomän som heter "artiklar" och en A-entry som pekar mot 206.189.1.58, är det möjligt?

Jag har förberett för att man ska kunna skicka med de där parametrarna för pnr genom iframe:n (https://retrosearch.recraft.me/?_inputs_&subscriber=1922) vilket betyder att det bör fungera redan nu (om man modifierar index-sidan på alternativ.nu/artiklar för att plocka upp någon sådan parameter, vilket du kan prova att göra om du vill redan nu, men vilket betyder att den sidan behöver plocka upp en parameter och använda denna i src för iframe:n).

recraft-ou commented 4 years ago

Jag kan testa lite snabbt att bara "hårdkoda" det där token-värdet till att börja med?

TidningenAter commented 4 years ago

Jag kan testa lite snabbt att bara "hårdkoda" det där token-värdet till att börja med?

Tja, kan det funka på riktigt med det sedan tror du?

TidningenAter commented 4 years ago

För att prova det alternativet så får jag be dig om att göra en DNS inställning på Loopia för alternativ.nu, lägga till en subdomän som heter "artiklar" och en A-entry som pekar mot 206.189.1.58, är det möjligt?

Jag tror jag måste be fs-data om detta, på måndag.

TidningenAter commented 4 years ago

För mig funkar det fortfarande bra att lägga saker i varukorgen med Chrome, men man hamnar därefter på https://retrosearch.recraft.me/ Bäst är om man stannar på www.alternativ.nu/artiklar, och inte heller kommer till butiken alls, eftersom du nu lagt till en så bra "Till kassan"-knapp

Även om man får till så man stannar kvar på sidan nu så laddas den ändå om, så att all filtrering och ev angivet p-nr försvinner, går det att göra något åt?

recraft-ou commented 4 years ago

Nu börjar jag tro att detta är kopplat till något som händer på shopcart-sidan; när man klickar på bilden till försättsbladet så tappas inte inställingarna, men köplänken gör att filterinställningarna tappas... Ser du samma sak? Och är det någon skillnad vad gäller Safari nu? Om det är så att det är köplänken som gör något på shopcart-sidan så kanske det går att göra en lösning på php/PS-sidan med en wrapper där som man länkar till....

TidningenAter commented 4 years ago

Japp, det verkar stämma. Funkar så för mig också både i Chrome och Safari. I butikens inställningar hittade jag "Redirect after adding product to cart (Only for non-AJAX versions of the cart)

Previous page är valt så den laddas väl om för att korgen ska uppdateras. Väljer jag Cart summary kommer man till kassan, inom iframen, och om man då backar i webbläsaren så är filtreringsinställningarna intakta. Inställningen ska ju gälla non-Ajax carts, men jag har ajax aktiverat och om jag tar bort det blir ingen skillnad.

TidningenAter commented 4 years ago

Lösning skulle ju kunna vara att välja cart summary istället och antingen lägga till en back-pil i kassan (blir dock fult i iframen) eller öppna länken i en ny flik med _blank som man får stänga, om det inte kan göras automatiskt.

recraft-ou commented 4 years ago

Flera funderingar:

Här är en tråd som diskuterar samma eller något väldigt snarlikt:

https://www.prestashop.com/forums/topic/15162-redirect-to-custom-page-after-adding-product-to-cart/

Om nu inte "previous page" hos PS automatiskt bestäms exv av https://en.wikipedia.org/wiki/HTTP_referer när PrestaShop Cart:en avgör vart den ska redirecta så .... vad tror du om denna idé?

// om man lyckas redirecta till http referer (och den är korrekt satt) https://www.alternativ.nu/butik/cart-wrapper?add=1&id_product=261&token=c1359edf2c70acccc6a966b718058cf0

// om callback länk anges explicit https://www.alternativ.nu/butik/cart-wrapper?add=1&id_product=261&token=c1359edf2c70acccc6a966b718058cf0&referrer_redirect=xxxxxxxxx(dvsurltillsidansomharlänken)

Edit: Jag hittade ett inlägg nu som säger att PS cart:en med "Previous page" inte avser sidan som länkat in utan en specifik sida i PS, dvs http referer används inte alls. Därmed en php-wrapper kanske vore något att prova? Den borde inte störa befintlig funktionalitet. I värsta fall kunde man ändra i cart.php men det känns lite som att det lättare kan påverka andra bitar av PS.

TidningenAter commented 4 years ago

Jag har försökt sätta mig in i det du talar om men är rädd att jag har för lite kunskap för att ge en vettig synpunkt. Om det är mycket jobb att prova ditt förslag borde vi kanske snabbt överväga hur viktig funktionen är eller om man kan göra en workaround tills vi vet om vi ska använda PS på sikt.

Vet inte om det kan vara relevant men:

TidningenAter commented 4 years ago

Vi kanske ska byta namn på denna tråd eftersom problemet med köpknappen är fixat och det nu väl bara handlar om att sidan laddas om så filtreringen försvinner.