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

Sales order overdue #1177

Merged
merged 7 commits into from Dec 18, 2020

Conversation

SchrodingersGat
Copy link
Member

Add "overdue" concept for sales orders

@SchrodingersGat SchrodingersGat mentioned this pull request Dec 18, 2020
3 tasks
@SchrodingersGat SchrodingersGat merged commit 7560b7e into inventree:master Dec 18, 2020
@SchrodingersGat SchrodingersGat deleted the sales-order-overdue branch December 18, 2020 09:06
@eeintech
Copy link
Contributor

eeintech commented Dec 29, 2020

@SchrodingersGat Is a date picker widget supposed to pop-up when clicking inside the "Target Date" field in the Create Sales Order form? I'm not able to get it to function and can't even change the date manually, which prevents creation of new sales order.

Could it be crispy form related? Or browser?

Also, we should make sure this field does not prevent the form validation, which is the case right now.

@SchrodingersGat
Copy link
Member Author

I did not realize that this field was required for the form validation, that's a bit of a blooper

With regards to the existing date picker widget:

Chrome

image

Firefox

image

What browser are you using?

Also I have an outstanding issue to implement this "properly":

#1170

@eeintech
Copy link
Contributor

eeintech commented Jan 2, 2021

@SchrodingersGat I found out that you need to click on the calendar icon on the right hand side to get the date picker widget to pop-up (this icon is hard to see with dark theme on), I was just desperately clicking anywhere in the field... I'm using Chrome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants