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.

Sign up for GitHub

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

Jump to bottom

Make sure ProcessInstance is not override by concurrent REST api call #137

Closed
teosarca opened this issue Feb 16, 2017 · 1 comment
Closed

Make sure ProcessInstance is not override by concurrent REST api call #137

teosarca opened this issue Feb 16, 2017 · 1 comment
Assignees
@metas-rc
Labels
branch:master type:bug
Milestone
2017-08

Comments

@teosarca
Copy link
Member

Test case:

  • open Receive HUs process
  • delete QtyTU (don't press enter or anything else)
  • type a number in QtyTU (don't press enter or anything else)
  • quickly click on "Save configuration" flag
  • sometimes the changed QtyTU is not taken, and you can quickly remark that back in quick actions.

image

In quick actions, the first action is the "Receive to ". You can spot that the quantity TU was not changed in it's name. In the example below, we have 3 x G2.
image
teosarca added a commit that referenced this issue Feb 16, 2017
@teosarca
#137 introduced ProcessInstance.version; enforce it on checkin
f31d0f0
teosarca added a commit that referenced this issue Feb 16, 2017
@teosarca
#137 Execution.prepareNewExecution(); refactor
33427ad
teosarca added a commit that referenced this issue Feb 16, 2017
@teosarca
#137 ProcessRestController retry on InvalidDocumentVersionException
fbcb815
@teosarca teosarca added this to the 2017-08 milestone Feb 17, 2017
@metas-ts metas-ts changed the title Make sure ProcessInstance is not override by concurrent RES api call Make sure ProcessInstance is not override by concurrent REST api call Feb 18, 2017
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Feb 18, 2017
@metas-ts
Update ReleaseNotes.md
0d0c335 
[#227](metasfresh/metasfresh-webui-frontend-legacy#277) Processed HU in Material Receipt Workflow
[#137](metasfresh/metasfresh-webui-api-legacy#137) Make sure ProcessInstance is not override by concurrent REST api call
[#294](metasfresh/metasfresh-webui-frontend-legacy#294) Leave Location editor with no entry
@metas-rc metas-rc self-assigned this Feb 21, 2017
@metas-rc
Copy link
Member

Results of IT1

Tested on http://w101.metasfresh.com:8090

I repeated the test and the config was always saved. OK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@metas-rc metas-rc

Labels
branch:master type:bug
Projects
None yet
Milestone
2017-08
Development

No branches or pull requests
2 participants
@teosarca @metas-rc