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

New Warehouse is not saveable #259

Closed
teosarca opened this issue Mar 18, 2017 · 1 comment
Closed

New Warehouse is not saveable #259

teosarca opened this issue Mar 18, 2017 · 1 comment
Assignees
@metas-mk
Labels
branch:master branch:release type:bug

Comments

@teosarca
Copy link
Member

Is this a bug or feature request?

bug

Which are the steps to reproduce?

  • create a new Warehouse
  • will never be saved because C_Location_ID is not filled.
    NOTE: that field is actually a legacy field which is set from C_BPartner_Location.C_Location_ID on before save (via model interceptor)... but we are not reaching that point.
teosarca added a commit to metasfresh/metasfresh that referenced this issue Mar 18, 2017
@teosarca
metasfresh/metasfresh-webui-api-legacy#259 fix New Warehouse is not s…
92010a7 
…aveable

solution: have a callout which is seting the legacy C_Location_ID from
C_BPartner_Location.C_Location_ID
@metas-lc metas-lc mentioned this issue Mar 18, 2017
Closed
@metas-mk metas-mk self-assigned this Mar 18, 2017
metas-mk added a commit to metasfresh/metasfresh that referenced this issue Mar 18, 2017
@metas-mk
Update ReleaseNotes.md
c91e2ce 
Add to the upcoming release. metasfresh/metasfresh-webui-api-legacy#259 New Warehouse is not saveable
@metas-mk
Copy link
Member

Tested. Works nicely as required. Closing.

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:bug
Projects
None yet
Milestone
No milestone
Development

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