Nexxus / NSK

Nexxus Stock Keeping
https://nexxus.eco/
GNU General Public License v3.0
14 stars 9 forks source link

[6h00m] Create and edit Purchase Orders #52

Closed nebbii closed 5 years ago

nebbii commented 6 years ago

Currently products are made with a Create form, eventually this will be done with a flexible order form where every product is bound to an order.

Terminology is viewed from a company's perspective: Purchase form is for things that the company purchases, sales is for what the company sells.

ronalddebakker commented 5 years ago

In de kleuren is ook een green Green: #009649 opgenomen. Dat is echter geen Copiatek kleur. Deze is er eens in gekomen toen Ben aan de search had gewerkt. De "Dark Green en Blue" zijn wel goed. Wellicht kan er nog met grijstinten worden gewerkt.

ronalddebakker commented 5 years ago

ik mis de functie om te kiezen voor ordernummer zelf invoeren (bijv. een ordernummer van de leverancier overnemen) of de keuze er 1 te laten genereren. Wellicht in de vorm ymd00001 en oplopend

ronalddebakker commented 5 years ago

Voordat attributen bij een product ingevuld kunnen worden, moet je eerst atttributen aanmaken in het Admin deel. Vervolgens die attributen toewijzen aan product types, ook in Admin.

Ik had verwacht templates te kunnen maken en deze template te kunnen toewijzen aan een product. Bij een bestaand product kan ik dan de template ook in een later stadium toe/invoegen.

Maar wellicht is het nu ook al een template omdat de attr. per producttype wordt gemaakt.

ronalddebakker commented 5 years ago

De PO moet aan een locatie kunnen worden toegewezen anders dan de locatie van degene die het invult.

jorr-it commented 5 years ago

PO krijgt reeds locatie toegewezen en dat gebeurt nu onderwater, overeenkomstig locatie van gebruiker. Als een gebruiker met een andere locatie inlogt, dan krijgt hij die PO niet te zien. Dat kan je testen.

Wat je impliciet lijkt te vragen, is dat een gebruiker voor meerdere locaties kan werken. Dat kan nu nog niet. In de voorcalculatie is die wens benoemd op rij 51: "Rollen die meerdere locaties mogen inzien". Als die rol geimplementeerd is, zullen diverse objecten een Locatie selectieveld krijgen.

Aanmaken van ordernr neem ik mee in issue #85 c.q. voorcalculatie rij 58 en 59.

Admin van product types en attributen kunnen verbeterd worden, als besproken, maar is nog niet gepland in voorcalculatie.

Groene kleur is nu allemaal overeenkomstig huisstijl als het goed is, maar ik kan hier of daar nog een waarde zijn vergeten.

ronalddebakker commented 5 years ago

Rij 51 klopt inderdaad. Dit is goed.

Wat ik niet duidelijk maakte in mijn comment, is dat eigenlijk de manager (ANWB) die een ophaalactie (PO) aanmaakt, deze moet kunnen toewijzen aan een locatie. Dit in verband met de mobiele app, zodat de ophaalactie (PO) alleen zichtbaar is voor de vrijwilligers van deze locatie.

Als ik (Copiatek) een PO aanmaak, dan wil ik deze ook kunnen toewijzen aan een locatie. Voor mij zal een locatie bijvoorbeeld ook een afleveradres (magazijn, kantoor, elders) kunnen zijn.

jorr-it commented 5 years ago

Ja dat is duidelijk. Ik ga dat verzorgen als ik ga starten met rij 51.

jorr-it commented 5 years ago

ALTER TABLE aorder ADD order_date DATETIME NOT NULL;

ronalddebakker commented 5 years ago

Ik kom toch niet uit het attributen deel. Volgens mij moeten het toch templates worden omdat een HDD of SSD ook nog diverse attributen krijgen. Het kan komen doordat een ander issue nog gemaakt moet worden of dat ik nog niet snap hoe het werkt of dat er een knop niet goed werkt/ontbreekt of iets anders wat ik nu niet kan overzien. Na 19 oktober kunnen we het doornemen.

jorr-it commented 5 years ago

Datum: Sat 29-9-2018 13:00
Gewerkt: 2h30m
Toelichting: #52
Dit is een automatisch bericht. Aan dit bericht kunnen geen rechten worden ontleend. Ref: 51821380

jorr-it commented 5 years ago

Datum: Sun 30-9-2018 09:00
Gewerkt: 3h30m
Toelichting: #52
Dit is een automatisch bericht. Aan dit bericht kunnen geen rechten worden ontleend. Ref: 51831967