Pregled projekta

[inesmersak]

• Za lažjo uporabo bi prosila, da v README.md dodate vse pakete, ki morajo biti instalirani, da uporabnik lahko sploh zažene vašo aplikacijo (shinydashboard, shinyjs, dbplyr, ...). V nasprotnem primeru mora človek pognati vašo aplikacijo kar nekajkrat, da ugotovi, katere vse pakete mora instalirat.
• V kodi imate še vedno sourcano datoteko auth.R: https://github.com/Charisa/Borza-OPB/blob/660c0c4f85167ef6461ffd7e5660174ef6fe71cf/server/server.R#L5 do katere navadni uporabnik nima dostopa -- to zamenjajte z datoteko auth_public.R. Pri tem poskrbite, da ima uporabnik javnost na bazi, ki jo uporabljate, prave pravice (za začetek mora imeti dovoljen CONNECT na bazo, potem pa verjetno še SELECT na vse tabele in INSERT na nekatere).

Trenutno mi vaša aplikacija ne deluje: buy / sell ne naredi ničesar, Wallet podstran je prazna, nikjer ne vidim možnosti registracije, čeprav vidim, da ste kodo napisali -- verjetno je to posledica tega, da javnost nima dostopa do baze? Ko to popravite, bom nadaljevala s pregledom.

[PirnovarA]

Sem dodal zahtevane pakete v readme. Sedaj sourca auth_public.R, ima pa tudi zahtevana pooblastila (to jih je ze prej imel). Ostalo pa dela. Sem preveril na novo zagnanem praznem R-u, in dela brez problema (prva stran je sign in stran, s katere se gre lahko tudi na sign up). Problem je bil, da smo pozabili mergati z delovnim branchem.

[inesmersak]

Vidim, v čem je bil problem: commite ste imeli shranjeno samo lokalno (ko sem pregledovala, je bil zadnji commit narejen 7. julija), niste jih pa še pushali, tako da sem imela vašo staro kodo.

Do vikenda dobite še preostale popravke, če bo kakšen.

Edit: ravno, ko sem napisala komentar, sem videla se tvoj edit glede merga :)

[inesmersak]

• Ko depositam ali withdrawam, se mi pojavi error v dveh modal boxih, ki pa po dveh sekundah zgine, nato se prikaže stanje na računu (ki je pravilno). Priložen je screenshot, pa še error, ki ga dobim v konzoli:

  <simpleError in eval_bare(x, .env): argument "ID" is missing, with no default>
  Warning: Error in tryCatch: object 'balance' not found
  96: tryCatch [serverFunctions.R#111]
  95: check.wallet.balance [serverFunctions.R#99]
  92: renderText
  91: func
  78: origRenderFunc
  77: output$walletStatusFiat
  1: shiny::runApp
  <simpleError in eval_bare(x, .env): argument "ID" is missing, with no default>
  Warning: Error in tryCatch: object 'balance' not found
  96: tryCatch [serverFunctions.R#111]
  95: check.wallet.balance [serverFunctions.R#99]
  92: renderText
  91: func
  78: origRenderFunc
  77: output$walletStatusFiatModal2
  1: shiny::runApp

  

• Search okvir zgoraj desno na podstrani Exchange se nekaj sekund po tem, ko vpišem query, sam od sebe resetira: izbrano imam npr. Devon Rex pasmo, vtipkam i v Search, lepo sfiltrira (ne najde nobenih zadetkov), preštejem do 8 in Search box kot tudi zadetki se resetirajo nazaj na default (Search box je prazen, prikazani so vsi zadetki). Isto se dogaja tudi pri searchu po pasmah, ceni in številu, pa tudi pri izbiri, koliko zadetkov na stran prikazati.
• Morda bi bilo smotrno imeti pri pasmah še opcijo All cats, ki prikaže vse listinge, potem bo search po pasmah sploh imel smisel.
• Najprej sem dodala listing za 1 mačko pasme Ragdoll po ceni 500, potem pa kupila 2 mački pasme Ragdoll. Pri tem je aplikacija pravilno zračunala, da moram plačati 1500 (v bazi je bil samo še en listing za to pasmo mačke in sicer po ceni 1000). Ko grem na Transaction history, dobim tole: Torej zabeleženo je, da sem kupila in prodala mačko po 500, in da je nek drug user prodal mačko za 1000, medtem ko ni zabeleženo, da sem jaz tista, ki sem kupila to mačko za 1000. (Verjetno je problem v https://github.com/Charisa/Borza-OPB/blob/95998d471b81232b2ed3a00fb25a08439d2e0e25/server/serverFunctions.R#L238-L244).
• Ko poskušam kupiti 0 mačk, je gumb Confirm Buy order lepo disablan. Če pa poskušam kupiti kakšno mačko, za katero ni listingov (npr. Persian), pa lahko stisnem na Confirm gumb in aplikacija se sesuje:

  <simpleError in `[.data.frame`(tabela_cen, 2): undefined columns selected>
  <simpleError in `[.data.frame`(tabela_cen, 2): undefined columns selected>
  <simpleError in `[.data.frame`(tabela_cen, 4): undefined columns selected>
  Warning: Error in tryCatch: object 'status' not found
  74: tryCatch [serverFunctions.R#330]
  73: execute.buy.order [serverFunctions.R#287]
  72: observeEventHandler [C:\Users\Ines\Documents\faks\opb\Borza-OPB-master\server/server.R#222]
  1: runApp

• Če poskušam dodati sell listing za 1.5 mačke, se aplikacija ne pritoži, temveč sprejme listing in količino zaokroži na 2.
• Če se registriram s šumnikom v usernamu, aplikacija to težko prebavi: Welcome, inesmeršak. Lepo bi bilo, da pri registraciji pri okenčku username uporabnika opozorite, naj si izbere tako ime, ki vsebuje le znake angleške abecede in številke (recimo). Nato še preverite, da je uporabnik res izbral ime, sestavljeno le iz dovoljenih znakov, in ga v nasprotnem primeru opozorite. (Prav tako je lepo polja registracije, ki so obvezna, označiti z zvezdico.)
• V datoteki db_creation.R sem opazila, da imate sicer posebej napisano funkcijo za grantanje pravic uporabniku javnost, vendar pa potem vseeno pokličete navadno grant funkcijo tudi za uporabnika javnost; https://github.com/Charisa/Borza-OPB/blob/95998d471b81232b2ed3a00fb25a08439d2e0e25/db_creation.R#L40 To ni najbolje, saj lahko pomeni, da lahko kdorkoli s tem uporabniškim imenom in geslom uporablja ne samo INSERT, temveč tudi ALTER in DELETE na vaših tabelah. Bolj smiselno bi bilo, da date uporabniku javnost samo pravice za INSERT in UPDATE na tistih tabelah, kjer je to potrebno.

[PirnovarA]

• Error odstranjen

• Kljub trudu in pravi kodi search noce it stran (po dokumentaciji in StackExchange-u bi mogu it). Na vsakih 10 sek se updajta, tako da se zato resetira.

• Preimenovali smo vsebino action-a, da je bolj razumljiva (prej je delala pravilno, le da smo sedaj se izognili dvoumnosti poimenovanja-iz sold v sold to you in iz bought from you)

• Ce se hoce kupit macke, ki nimajo listinga, se ne zrusi vec, se pokaze obvestilo

• Obvestilo, da mora biti kolicina celostevilska, dodano

• Odstranjena moznost prijave s sumniki.

• Dodane druge pravice za javnost

[MarvinHerzog]

• Pri Deposit lahko uporabnik vnese negativno število. Preko tega lahko stanje na računu postane negativno. V primeru da vnesem zelo veliko število (100000000000000000000000), moje stanje prikaže kot nekaj manjšega (99999999999999991618642), čeprav aritmetika še vedno deluje, kot da je na stanju prvi znesek. Če pri Deposit vnesem string, potem se stanje (pravilno) ne spremeni, vendar dobim sporočilo "Deposit successful".

• Na Exchange lahko kupim mačko, ki sem jo sam objavil. Tega ni potrebno nujno spreminjati, vendar je možna alternativa, da dodate cancel gumb pri uporabnikovih objavah.

• Morda bi bilo bolje, da odstranite posodabljanje Exchange tabele vsakih 10 sekund in namesto tega posodobite tabelo le ob določenih uporabnikovih dejanjih, npr. ko mačko kupi, mačko proda, izbere drugi tip mačke ali pa ko izbere zavihek Exchange. S tem se lahko izognete resetiranju polj Search, entries in trenutnega izbora strani tabele.

• Vidim, da ste spremenili dodeljevanje pravic v db_creation.R, vendar ima uporabnik javnost trenutno še vedno vse pravice (te lahko preverite na vaši bazi pod Tables>tabela>Privileges). Uporabite revoke, da omejite ustrezne pravice.

Razen tega zgleda z moje strani vse v redu.

[PirnovarA]

• Popravil

• Verjetno ne bi vzelo ogromno casa, ampak trenutno delamo diplomo (mogoce do cetrtka naredimo se to)

• Ni vec searcha, posodablja ob prodaji, nakupu, spremembi macke, in na 60 sec

• Odstranil pravice, sedaj bi mogla met samo dodeljene

[jaanos]

Sem dodal še povezavo na repozitorij predmeta. Opozoril bi sicer še na eno malenkost:

https://github.com/Charisa/Borza-OPB/blob/684b144388be596feb72421bc3a6d8d1bea8a07a/server/server.R#L354-L355

Tukaj ste najbrž želeli input$walletDepositInput, da bo mogoče nalaganje sredstev.

Če boste še kaj spreminjali, mi prosim javite, da posodobim povezavo.