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

"Not saved yet" info in REST-API #191

Closed
metas-mk opened this issue Feb 25, 2017 · 3 comments
Closed

"Not saved yet" info in REST-API #191

metas-mk opened this issue Feb 25, 2017 · 3 comments
Assignees
@metas-mk
Labels
branch:master branch:release type:enhancement
Milestone
2017-09

Comments

@metas-mk
Copy link
Member

Is this a bug or feature request?

Feature Request

What is the current behavior?

Currently no, or only debug information is existing via Rest-API for windows about the save status (saved, not saved).

Which are the steps to reproduce?

Check any sample window.

What is the expected or desired behavior?

Provide data about the save status, so the frontend can react on that and display this information to the user.
@metas-mk metas-mk added this to the 2017-09 milestone Feb 25, 2017
@metas-mk
Copy link
Member Author

Linked to Frontend Issue: metasfresh/metasfresh-webui-frontend-legacy#437

teosarca added a commit that referenced this issue Feb 25, 2017
@teosarca
#191 JSONDocument.saved flag
74cbe6b
@teosarca
Copy link
Member

Note to IT: to be tested when frontend task is done

@teosarca teosarca removed their assignment Feb 25, 2017
metas-mk added a commit to metasfresh/metasfresh that referenced this issue Feb 27, 2017
@metas-mk
Update ReleaseNotes.md
beabf02 
Add [#191](metasfresh/metasfresh-webui-api-legacy#191) to this week's release 4.58.57
teosarca added a commit that referenced this issue Feb 27, 2017
@teosarca
#191 bugfix saveStatus/validStatus not updated correctly; refactor
286fdb2
teosarca added a commit that referenced this issue Feb 27, 2017
@teosarca
#191 improve error reporting
e1a927a
teosarca added a commit that referenced this issue Feb 27, 2017
@teosarca
#191 improve error reporting
90de886
teosarca added a commit that referenced this issue Feb 27, 2017
@teosarca
#191 ProcessInstance refactor - use isSaved()
ece8640
@metas-mk
Copy link
Member Author

metas-mk commented Mar 1, 2017

Tested w/ Product window. Initially saved status: false. Edited 2 mandatory fields (1 remaining untouched). saved status: false. edited the remaining mandatory field, saved status: true.
Works as required. Won't wait for frontend task. Can be tested independently.
Closed and moved to done.

@metas-mk metas-mk closed this as completed Mar 1, 2017
@metas-mk metas-mk self-assigned this Mar 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@metas-mk metas-mk

Labels
branch:master branch:release type:enhancement
Projects
None yet
Milestone
2017-09
Development

No branches or pull requests
3 participants
@teosarca @metas-ts @metas-mk