search

CORE-POS / IS4C

Cooperative Operational Retail Environment
http://www.core-pos.com
GNU General Public License v2.0
64 stars 44 forks source link

When prompt for member number comes up allow for a scan of the member card #58

Closed DonaldGonzalez closed 11 years ago

DonaldGonzalez commented 12 years ago

Currently, if a cashier forget to scan a member card and hits total then there is a pop-up box that reminds the cashier to enter a member number. The same prompt only allows for input through the keyboard but does not allow for the scan of a member card. This should input box should allow for the scan of a member card. This has been added to phase 3.

gohanman commented 12 years ago

This fix works for me: a2d1a075e6a3d647a1e87448b3e087f1592702af

DonaldGonzalez commented 12 years ago

@joelbrock Can you pull this commit to our test lane?

DonaldGonzalez commented 12 years ago

@joelbrock Have you pulled this commit down to our test lane yet? @pierced Do you think that we should have Joel pull all updates to our test lane that we have missed since we relocated?

pierced commented 12 years ago 
I do.  Joel, can you do this and let us
  know when it has been done and what has been pulled down so we can
  test?
  Thanks

Don Pierce Information Systems Manager Harvest Co-op Markets Cambridge and Jamaica Plain, MA

617-661-1580 ext.123

  Donald Gonzalez wrote:

  @joelbrock Have you pulled this
    commit down to our test lane yet? @pierced
    Do you think that we should have Joel pull all updates to our
    test lane that we have missed since we relocated?

    —
    Reply to this email directly or view
      it on GitHub.
joelbrock commented 12 years ago

Merged changes and testlane updated with latest from CORE.

On Fri, Sep 28, 2012 at 2:07 AM, pierced notifications@github.com wrote:

I do. Joel, can you do this and let us know when it has been done and what has been pulled down so we can test? Thanks

Don Pierce Information Systems Manager Harvest Co-op Markets Cambridge and Jamaica Plain, MA

617-661-1580 ext.123

Donald Gonzalez wrote:

@joelbrock Have you pulled this commit down to our test lane yet? @pierced Do you think that we should have Joel pull all updates to our test lane that we have missed since we relocated?

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/CORE-POS/IS4C/issues/58#issuecomment-8968551.

DonaldGonzalez commented 12 years ago

@joelbrock now the scale at the lane will not scan anything. The scale does not beep any longer either. When I try to scan an item all I can see is that it reloads at the tab but nothing actually beeps or appears on the screen including the commit you just merged for this issue.

DonaldGonzalez commented 12 years ago

So now the scale is scanning. The member card will scan into a member search box, the only problem is that when the member card is scanned it is reading the entire UPC. When I scan my member card the scanner reads 7302152071, it should only read the last 6 digits, 152071. Since the scanner reads the entire UPC, CORE tells me that the member is not found, this happens when I scan the card and type in the entire UPC. But when I type in the last 6 digits, it looks me up just fine. I think this is how we have had it working from the beginning. We also need the scanner to beep when a card is scanned and a member is found, we need an error beep when a member record cannot be found.

gohanman commented 11 years ago

a6bc556cb7fe9c5ad9b1ea0face24834f56374e3

Hooray for housekeeping. Stuff that still needs doing gets noticed.