search

kilbot / WooCommerce-POS

:bangbang: All development now at https://github.com/wcpos.
http://wcpos.com
GNU General Public License v3.0
353 stars 125 forks source link

Barcode scanner does not allow bulk scnanning #244

Open rleegwater opened 3 years ago

rleegwater commented 3 years ago

It's not possible anymore to preform multiple scans after each other. After each scan we have to press Y to get the cursor back in the scanfield.

We have the lates Version installed: WC Version 5.1.0 PHP Version 7.4.16 WC REST API API is active

Honeywell scanners: Voyager™ 1200g-2 (streepscanner) Quantum™ E MS7120 (P)38-3 (orbitscanner)

Anyone has this problem too?

mjavs81 commented 3 years ago

It's not possible anymore to preform multiple scans after each other. After each scan we have to press Y to get the cursor back in the scanfield.

We have the lates Version installed: WC Version 5.1.0 PHP Version 7.4.16 WC REST API API is active

Honeywell scanners: Voyager™ 1200g-2 (streepscanner) Quantum™ E MS7120 (P)38-3 (orbitscanner)

Anyone has this problem too?

same here..need to position the cursor in the search box after every scan..hope it gets resolved soon..

sbotman commented 3 years ago

@kilbot Having the same issue. Could you please add something like: e.ui.searchField.focus(); after e.triggerMethod("clear"), e.ui.searchField.blur() so it sets the focus on the field. Tnx!

kilbot commented 3 years ago

Hi @sbotman, thanks for the PR! I am currently rewriting the plugin using React Native over at https://github.com/wcpos. I am getting very close to the initial beta release so I'm not sure if I will package a 0.4.x update. Did you manage to build the JS files with your change?

sbotman commented 3 years ago

Hi @kilbot, I'm working thru the doc and setting up my machine to build the complete project, but running into issues here and there. Lately we are running into different issues with the POS site, products that cannot be found and calls that are not executed while updating the search field. I have no idea why, because the site is fast and everything is responding under 1 sec. I know you are not active on this codebase and doing the rewrite, but in the meantime I need to support the girls in the store so searching for some solutions...

kilbot commented 3 years ago

I haven't built the JS files myself for years, so there could be outdated libraries and all sorts of trouble. I don't want you to waste your time.

How urgently do you need this change? I can probably update the JS bundle if you need this right now. But if you can wait a week you can help test the new version which will hopefully solve a lot of your pain points.

sbotman commented 3 years ago

Well, we use the POS on daily basis and they are having issues the last weeks, but I'm sure that one extra week should be doable. Would be more than happy to help you with testing the new code base.

sbotman commented 3 years ago

Managed to compile the JS bundle and update it for me personally. Had some issues with the before script of the tests, so had to skip this one. The error was not related to my change, so I must be missing something: got some type errors like:

  2) entities entities/orders/model.js "before each" hook for "should be in a valid state":
     TypeError: Cannot read property 'channel' of undefined
rleegwater commented 3 years ago

@sbotman Hi Sander, where you able to fix this problem? We still have this problem and it's getting really annoying.... Sinterklaas verkoop is almost starting and this will generate always a lot of shop customers....

m-zat commented 3 years ago

Hi @rleegwater the PR #248 that @sbotman opened solves the issue. You just need to apply that to the 0.4 branch and rebuild the JS.