Fix completeSalesOrderShipment typo in sales_order.js

[martonmiklos]

Fixes #4906

[SchrodingersGat]

Nice catch @martonmiklos - we really need to get some CI testing on the front end code ASAP

[martonmiklos]

Nice catch @martonmiklos -

Credits goes to @jelgersma

we really need to get some CI testing on the front end code ASAP

What is the plan for that (what technology do you prefer)?

[SchrodingersGat]

What is the plan for that (what technology do you prefer)?

I don't really have enough knowledge about the unit testing frameworks available to make a call on that! Have to read some more. Can you make any recommendations?

[wolflu05]

I would say some linting would be enough for the first, as there is anyway the plan to switch to react with typescript?

Linting, eg with eslint would cover the undefined function references.

[SchrodingersGat]

We already do run eslint - https://github.com/inventree/InvenTree/blob/master/.eslintrc.yml - so we should look into why it did not catch this incorrect function