search

jmueller17 / Aixada

Aixada helps self-managed consumption cooperatives to organize their flow of products, money, and information.
Other
33 stars 37 forks source link

Comanda acumulativa, filtre per proveïdor #236

Closed dmanubens-zz closed 5 years ago

dmanubens-zz commented 5 years ago

Quan fem una comada acumulativa, es mostra un llistat amb tots els productes disponibles de tots els proveïdors. En la comanda normal es mostren només els del proveïdor sel-leccionat. Ens agradaria poder consultar els productes disponibles en comanda acumulativa corresponents a un sol proveïdor.

situplastik commented 5 years ago

Bones,

A la notra cooperativa, justament també volíem el mateix. Vam fer un fork del projecte i entre les modificacions que vam fer (tenim un funcionament diferent al que la aixada proposa), una va ser aquesta. La modificació va ser per jQuery (per no tocar procedures MySQL ni codi intern); no és el més òptim però funciona. Mostrem per php tots els productes (com ara) però tenim un select amb tots els proveïdors i filtra tant el llistat de productes de la esquerra com la "cistella" de la dreta.

Si només ho voleu vosaltres us podem pasar el codi que vam tocar per aquesta "issue" però si més gent ho vol puc mirar de fer un PR a tot el projecte. @jorix, @jmueller17 , què n'opineu?

Relacionat amb aquest tema, també vam posar un botó nou quan estas veient els productes d'un proveïdor, per poder passar tots els productes de cop a acumulatius, sense haver de clicar un per un.

Fins ara!

jorix commented 5 years ago

És un tema que té molt de sentit. M'agtadaria veure el fork.

Per altre part, he pensat diverses vegades que seria bo presentar la llista de dies en que hi ha oberta comanda en lloc del calendari. A la gent nova sempre s'ha d'expicar.

Si es mostres una llista de dies amb un item més per les acumulatives obertes, (deixant se ser la quarta pestanya)... Tindriem els mateixos filtres per les d'un dia que per les acumulatives... Volia provar aquesta via. Que us sembla?

El dg., 18 nov. 2018 22.09, Lluís Marfany notifications@github.com va escriure:

Bones,

A la notra cooperativa, justament també volíem el mateix. Vam fer un fork del projecte i entre les modificacions que vam fer (tenim un funcionament diferent al que la aixada proposa), una va ser aquesta. La modificació va ser per jQuery (per no tocar procedures MySQL ni codi intern); no és el més òptim però funciona. Mostrem per php tots els productes (com ara) però tenim un select amb tots els proveïdors i filtra tant el llistat de productes de la esquerra com la "cistella" de la dreta.

Si només ho voleu vosaltres us podem pasar el codi que vam tocar per aquesta "issue" però si més gent ho vol puc mirar de fer un PR a tot el projecte. @jorix https://github.com/jorix, @jmueller17 https://github.com/jmueller17 , què n'opineu?

Relacionat amb aquest tema, també vam posar un botó nou quan estas veient els productes d'un proveïdor, per poder passar tots els productes de cop a acumulatius, sense haver de clicar un per un.

Fins ara!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jmueller17/Aixada/issues/236#issuecomment-439726072, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDGDt5DddC74wDlJepwcYgMFwjz2bnUks5uwcyFgaJpZM4YoIyP .

situplastik commented 5 years ago

HTML & jQuery del select:

https://github.com/situplastik/Aixada/commit/c3e76a5d4e8bb588662b71e79dc4a8b0e44a554d (a partir de línia 458)

Veureu que fem un select nou amb un ID i per jQuery fem un llistat de providers per omplir-lo i creem un event onChange on anem iterant amb tots els elements dels dos llistats :)

Per altra banda, si es pogués fer el filtre que comentes a la primera pestanya sería genial. Entenc que et refereixes a tocar MySQL i procedures (a la nostra cooperativa no tenim molta experiència amb aquest tema) pel que no podem donar molt de suport. Però si, si es pogués fer així i poguessim treure la quarta pestanya sería genial. Mantindriem totes les funcionalitats i apart un tema menys a explicar als nous companys, que sempre els hi costa de trobar ;)

Fins ara!

jorix commented 5 years ago

A la branca a-preorder_filter he posat el que proposaré com PR.

Hi he posat:

Tinc pendent un detallet més, i el què trobeu si ho proveu (cal aplicar el aixada_queries_all.sql)

situplastik commented 5 years ago

Bones @jorix ,

He actualitzat el nostre repositori amb un nou commit amb canvis que fa poc hem fet (just fa unes setmanes), encara teniem pendent posar-ho al github.

Hem detectat que hi havia problemes fent una nova funció per AJAX, pel que hem canviat el sistema a un foreach jQuery que activa tots els productes d'un proveïdor com a acumulatius .

jorix commented 5 years ago

@dmanubens & @situplastik:

He creat una nova branca a-preorder_review_code ajuntant commits per temes per tal de facilitat la revisió del PR que proposaré sobre comandes acumulatives.

dmanubens-zz commented 5 years ago

Hola @jorix @situplastik Perdoneu, no he pogut mirar-ho fins avui. Em semblen molt bé els canvis que proposeu. Ara mateix no tinc entorn de testing perquè m'he canviat el portàtil i ho hauré de pujar al hosting directament. Tant bon punt estigui a master ho desplegaré, perquè ens interessa molt. Moltes gràcies! Typo--> aquí diria que és 'one provider': https://github.com/jmueller17/Aixada/compare/master...jorix:a-preorder_review_code#diff-3513c66a051d3889c51e5477d27ba6c5R901

jorix commented 5 years ago

Això ja està a master, aquest 'issue' ja el podeu tancar.

Mireu la nova plana wiki d'actualització.

jmueller17 commented 5 years ago

Perfecto.