Problem

Vid navigering till annan tab försvinner shortcut till varukorgen.

How to replicate....

Lägg något i varukorgen => Gå till homepage => navigera till order/profile page..

Homepage

Order

Expected behaviour

Jag förväntar mig att shortcut till varukorgen finns kvar, även på profile eller order-page

Anges fel pw/mail kommer notis upp och sedan ligger loading screen kvar över skärmen, omöjligt att komma därifrån

issue

Expo har för nuvarande ej support för hooks. ie. vi kan inte skriva hook-based components i vår react-native project ännu.

intro

Hooks är väldigt bra för functional stateful components, och rekommenderas över classbased components av react teamet. Majoriteten av react jag lärt mig är hook-baserad vilket gör mig något redundant på app-utvecklingen.

possibilites

Här skriver en utvecklare att det bör (kan) komma snart https://twitter.com/sjchmiela/status/1113327153263910912

Här är ett risky-fix
https://medium.com/@gwendall/using-hooks-with-expo-b9a655376447

how to fix:

• håll tummarna
• Låt mig skriva på webbsidan som tillåter hooks

Problem

Unwanted behaviour is being able to pull entire view,

To replicate

Navigate to payment-page, try pulling in the view,

Wanted behaviour

Only able to scroll through items, i.e my cart, now pull entire view.

Example pic dragging

Example pic not dragging

To fix.

Change some view-settings in App/pages/PaymentPage.js

Vid navigering direkt till 'Köp' läses inte orderID in vilket ger error " undefined (evaluating 'this.props.navigation.state.params.orderID)' "

Kan lösas i orderPage rad 6:
return ;

När man har betalat för en kaffe så kommer det upp en varning i appen som säger att man inte kan kalla på en "setState" funktion på en komponent som inte har laddat ännu.

Se bilder för bättre förklaring

Problem

Vid navigering från checkout till qr-koden rensas carten, vilket gör att checkout re-renders när vi navigerar från, här är en halvdålig bild.

Eventuell lösning

Lösningen har diskuterats en del, lite väl mycket för att dokumentera, men kortfattat kommer vi nog cleara-carten efter en liten delay...

Just nu finns inget sätt att veta om en order i vår backend är "aktiv", dvs när det går att hämta ut kaffet. Kanske borde lägga till en key active som är true eller false i en order?

Genom att ha nåt sånt kan man då t.ex. göra en koll med backenden när appen startas/efter en order placerats för o sedan visa en aktiv order i Köp-vyn.

I nuläget uppdateras inte propertien shop i carten när vi lägger till en kaffe från ett kafé. Den behöver uppdateras för att vi ska veta varifrån kunden köper sitt kaffe så att vi bland annat kan skapa kvitton på rätt sätt.

Det som behöver göras:

• Bestämma vartifrån shop hämtas - ska den hämta från ett kaffe-objekt eller ska en ny reducer-action skapas?
• Uppdatera shop i redux-statet (reducers.js)

Nuvarande ful lösning

const fixShopString = shop => {
let fixedString = shop.replace(' ', '%20');
fixedString = fixedString.charAt(0).toUpperCase() + shop.slice(1);
return fixedString;
}

Förbättringsförslag av @MarkusPettersson98 här: c15e083
och tidigare stängt issue här: #64

Radio buttons vill man helst inte ha i en mobilapp. För att välja egen mugg eller lånemugg tycker jag vi kör något i stil med:

Då får vi stora knappar som är lätta att klicka på och slipper även ett HELT klick.

Issue

Veras Café har ett mellanslag OCH en prick över e (é), vilket gör att namnet inte är samma som i webadressen för att hämta information om det.

Jämför

Veras Café

med det i url: https://expressobackend.herokuapp.com/api/getShop/Veras%20Caf%C3%A9

...Veras%20Caf%C3%A9

Fix

Filer som kan åtgärda detta i appen är
App/API/expressoAPI.js
och i backenden
Backend/API/api.js
och/eller
Backend/app.js

Om man tar sig till ett kafé ifrån kartan så dyker inte addressen upp i headern 🤔

Just nu finns det ingenting som visar vartifrån man håller på att handla, vilket kan bli lite förvirrande om man har lagt till någon/några kaffesorter i varukorgen och sedan navigerar till ett annat café.

Förslag

Visa iallafall i Alerten som kommer fram när man försöker att lägga till en kaffesort från ett annat café än det man håller på att handla ifrån.

issue

Vi har inte stöd för fler än 100 beställda kaffen för då ser det ut såhär.

how to fix:

fixa styling i
App/pages/components/order/OrderCartIcon.js

Just nu är det githubs ikon, ändra till något annat innan demot

Är man inne i en kafé-vy och klickar på bannern till varukorgen, sedan klickar tillbaka går man till framsidan istället för tillbaka till kaféet.

När en ny order läggs skriver det över alla tidigare kvtton, vilket tyder på fel i backend logiken.

Ändra betalningsmetod till kårkort

Kanske lägga på en timer som kollar om kvittot som visas fortfarande är aktivt?

Problem

Can enter payment page with empty cart

Replicate

Go to checkout, empty cart, go to payment.

Picture

Fix

Shouldn't be possible if #66 is reviewed

Det är skönt att vårstäda lite då och då, innan högen med technical debt växer sig lika stor som ens hög med lortig disk ..

Nåja, denna issue syftar till att identifiera, refaktorera och därmed optimera vår applikation, särskilt vad gäller att skjuta iväg onödiga datahämtningar. Vår cart är räddaren i nöden här, den vet nog mer om appen än @radjavi och @RobertZetterlund ihop.

Tips

En enkel optimering är att hitta och dra ner på användandet utav getShop, getShopById osv i programmet (där det inte behövs såklart).

Identifierade sub-optimala datahämtningar

• Checkout.js , rad 34 inhämtas data som redan finns i vår cart. Hämta shop från redux-state istället 98f0891

Riktig goldplating

Vore det inte fett att försöka sig på att cacha lite data? Så fort en användare navigerar till ett kafé skjuts i nuläget en request iväg, vilket snabbt adderas när användaren får beslutsångest och vill dubbelkolla vad alla kaféer har att erbjuda. Global hashmap någon?

Kvitton verkar inte lagras i firebase. Kan inte nå aktiva eller historiska kvitton

Problem

Vid navigering från HomePage till e.g. Biblioteket ändras bakgrundsfärgen

To replicate

Navigera från första sidan till biblioteket och notera färgskillnaden

pictures

startvy

cafevy

to-fix

Ändra bakgrundsfärger i olika containers för att få det konsekvent.
Se styleguide för rätt färg i wikin

Efter att varukorgsgenvägen tagits bort från huvudmenyn finns det skräpkod som behövs fixas. Detta kommer från främst Navigation.js, men även "efterföljande" fil(er)/klass(er) som importerats.

Ikoner som inte laddats in direkt gör så att appens andra komponenter hoppar omkring

Problem

När man hoppar in i en vy som inte laddat ikonerna händer det ibland att andra vy-objekt som har sin styling baserat på dem ändrar sin position när ikonen väl laddat klart.

Bilder

Notera (trots dåliga screenshots) att text hoppar runt beroende på om ikonen är renderad eller inte pga relationsbaserad styling.

How to replicate

Navigera till kaffevy på en "nystartad" demo. Notera problemet.

Även gäller detta bug vid första rendering av varukorgen. Undersök genom att lägg något i varukorgen och se att den röda (1) byter plats beroende på om ikonen laddats eller inte.

How to fix (I think)

Gör så att ikonerna är inneslutna i en <View> som har deras storlek, så att de endast fyller den istället för att ändra storlek vid laddning. Alltså hårdkoda views till att vara ikonstorlek. Detta bug finns lite överallt i appen, men förmodligen överallt där ikoner finns.