{"action":"created","issue":{"url":"https://api.github.com/repos/jekkos/opensourcepos/issues/125","labels_url":"https://api.github.com/repos/jekkos/opensourcepos/issues/125/labels{/name}","comments_url":"https://api.github.com/repos/jekkos/opensourcepos/issues/125/comments","events_url":"https://api.github.com/repos/jekkos/opensourcepos/issues/125/events","html_url":"https://github.com/jekkos/opensourcepos/pull/125","id":104585572,"number":125,"title":"#110 Change Overview from Sale page to be a daily view","user":{"login":"daN4cat","id":636048,"avatar_url":"https://avatars.githubusercontent.com/u/636048?v=3","gravatar_id":"","url":"https://api.github.com/users/daN4cat","html_url":"https://github.com/daN4cat","followers_url":"https://api.github.com/users/daN4cat/followers","following_url":"https://api.github.com/users/daN4cat/following{/other_user}","gists_url":"https://api.github.com/users/daN4cat/gists{/gist_id}","starred_url":"https://api.github.com/users/daN4cat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daN4cat/subscriptions","organizations_url":"https://api.github.com/users/daN4cat/orgs","repos_url":"https://api.github.com/users/daN4cat/repos","events_url":"https://api.github.com/users/daN4cat/events{/privacy}","received_events_url":"https://api.github.com/users/daN4cat/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-02T22:35:50Z","updated_at":"2015-09-03T00:23:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jekkos/opensourcepos/pulls/125","html_url":"https://github.com/jekkos/opensourcepos/pull/125","diff_url":"https://github.com/jekkos/opensourcepos/pull/125.diff","patch_url":"https://github.com/jekkos/opensourcepos/pull/125.patch"},"body":"I've done the changes as part of #110, but it's not completed and I need some help (see end of text).\r\n\r\n- The Sale/Overview is now using the sale_temp_table to speed up things\r\n- It fetches the daily transactions showing amounts, change and value per payment type (it's now also in the sale detail view as it used to be some years ago)\r\n- There is a total line for the amounts\r\n- There is a table with the totals per payment (the table column is dynamic according to the payment types)\r\n- The change due is removed from the cash total (it's a strong assumption but I would say safe one because there is no case you give a change to a credit card, it's not allowed by the credit/debit card companies as far as I am aware)\r\n- I removed the search box\r\n- general tidy up of code\r\n\r\nThere is an issue that I'm not able to solve easily because I'm not that familiar with UI stuff (html & js) - yet.\r\nI've left a checkbox to select the only_invoices but it doesn't trigger a view refresh.\r\nHowever if you put /1 or /0 at the end of the URL you can see that the selection works, so it's really a matter of js magic to get it to work properly.\r\nTo do a nice job all the search & suggest related stuff in that form between js & html should go and just change when the user ticks a checkbox.\r\nMaybe even the +/- drop down should be removed and leave the checkboxes visible all the time.\r\n\r\nIf you could help with that then I could add some check box to have a view for yesterday or some other date.\r\n"},"comment":{"url":"https://api.github.com/repos/jekkos/opensourcepos/issues/comments/137279770","html_url":"https://github.com/jekkos/opensourcepos/pull/125#issuecomment-137279770","issue_url":"https://api.github.com/repos/jekkos/opensourcepos/issues/125","id":137279770,"user":{"login":"odiea","id":10962177,"avatar_url":"https://avatars.githubusercontent.com/u/10962177?v=3","gravatar_id":"","url":"https://api.github.com/users/odiea","html_url":"https://github.com/odiea","followers_url":"https://api.github.com/users/odiea/followers","following_url":"https://api.github.com/users/odiea/following{/other_user}","gists_url":"https://api.github.com/users/odiea/gists{/gist_id}","starred_url":"https://api.github.com/users/odiea/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/odiea/subscriptions","organizations_url":"https://api.github.com/users/odiea/orgs","repos_url":"https://api.github.com/users/odiea/repos","events_url":"https://api.github.com/users/odiea/events{/privacy}","received_events_url":"https://api.github.com/users/odiea/received_events","type":"User","site_admin":false},"created_at":"2015-09-03T00:23:24Z","updated_at":"2015-09-03T00:23:24Z","body":"I tried your version and the other problem it appears to have is in the overview it does not show returns. I like the additional feature . Can a date picker(like the reports have) be used in your new feature."}}