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

bug #1179 #1339

Merged
merged 1 commit into from Dec 9, 2013
Merged

bug #1179 #1339

merged 1 commit into from Dec 9, 2013

Conversation

daamien
Copy link
Contributor

@daamien daamien commented Dec 6, 2013

if there's a delivery date (date_livraison) then use it instead of the validation date (date_valid) to check if we need to display a warning or not...

For example, if we are in Septembre and there's an order that was validated in june but its delivery date is set to november then a warning is not necessary, since there's nothing to do but wait for the order to be delivered.

see https://doliforge.org/tracker/?func=detail&aid=1179&group_id=144

if there's a delivery date (date_livraison) then use it instead of the validation date (date_valid) to check if we need to display a warning or not...

For example, if we are in Septembre and there's an order that was validated in june but its delivery date is set to november then a warning is not necessary, since there's nothing to do but wait for the order to be delivered.
@FHenry
Copy link
Member

FHenry commented Dec 6, 2013

Your code line was slipt on two lines (on a strange place), that why travis says it's not OK. check Files Changes tab

@daamien
Copy link
Contributor Author

daamien commented Dec 6, 2013

ah a stupid copy/paste error

My bad...

eldy added a commit that referenced this pull request Dec 9, 2013
@eldy eldy merged commit f363542 into Dolibarr:develop Dec 9, 2013
OGPayet pushed a commit to OGPayet/dolibarr that referenced this pull request May 11, 2021
…itué de la fiche utilisateur pour n'afficher que les utilisateurs internes dans la liste des responsables hierarchiques (à activer sur les entités pour etre prise en compte)
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

3 participants