Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automatised inventory system (#4) #15

Merged
merged 10 commits into from
Feb 12, 2017
Merged

Automatised inventory system (#4) #15

merged 10 commits into from
Feb 12, 2017

Conversation

kjagiello
Copy link
Member

@kjagiello kjagiello commented Feb 12, 2017

Checklist:

  • Add pdftotext to the deploy scripts.
  • Add pdftotext to the Travis configuration.
  • Mention the pdftotext dependency in the setup instructions.
  • Proper validation of the report file.

@kjagiello kjagiello added this to the 2.0.0 milestone Feb 12, 2017
@kjagiello kjagiello self-assigned this Feb 12, 2017
@kjagiello kjagiello requested a review from flaeppe February 12, 2017 13:07
@kjagiello
Copy link
Member Author

kjagiello commented Feb 12, 2017

We do not receive the proper quantities for some products (for example cutlery or ketchup). We should probably implement some kind of multiplier that can be defined on a per supplier product basis.

image
(We receive 100 ketchup packages in a single box).

@flaeppe
Copy link
Member

flaeppe commented Feb 12, 2017

Should we always assume it is 100?(as for now) Otherwise you can add the multiplier value on the webpage. If different suppliers have different amounts per package.

However manual filling of the multiplier amount is required to confirm the order then I guess.

@kjagiello
Copy link
Member Author

kjagiello commented Feb 12, 2017

No, we cannot assume that. I agree that multipliers should be configurable (defaulting to 1).

Some products are reported as bundles and their quantity and price
should be recalculated accordingly.
@kjagiello kjagiello merged commit 76c5caf into develop Feb 12, 2017
@kjagiello kjagiello deleted the feature/suppliers branch February 21, 2017 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants