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

A preorder review code #239

Merged
merged 9 commits into from Nov 30, 2018
Merged

Conversation

jorix
Copy link
Contributor

@jorix jorix commented Nov 29, 2018

This proposal is an revision of the code of cumulative orders.

Pick up some proposals already mentioned in #236, and the correction of some problems.

It also proposes the presentation of a date selector with open orders complementary to the calendar. For cumulative orders (if they exist) they are located in the first item of the selector. and the cumulative tabs disappear.
In the search tab there is a button to see everything of the day.

It has been proposed as a single PR, although with separate commits by themes.

@jmueller17 jmueller17 merged commit bd403d7 into jmueller17:master Nov 30, 2018
@jmueller17
Copy link
Owner

ya está!
moltes gracies :)

@jorix jorix deleted the a-preorder_review_code branch November 30, 2018 15:32
jorix added a commit to jorix/Aixada that referenced this pull request Mar 24, 2019
The error was introduced by me in jmueller17#239

Since there are no longer two tabs on the shopping cart, one for the date and another for the cumulative ones, there is no more double information in oper=commit of the order of the date and the preoder=true. Currently cumulatives are also worked as of date.

I did not see this `or date_for_order='1234-01-23'` in this `DELETE`, I'm sorry.
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