This commit is contained in:
2021-06-18 15:17:40 +00:00
parent 2e0d9fb801
commit 0ac783e020
2 changed files with 6 additions and 12 deletions

View File

@@ -386,7 +386,7 @@ CURRENTLY DOING: wo completion list
todo: notify event list table needs to show status of workorder (all fields) ideally
todo: notify subscription list table needs to show status of workorder (all fields) ideally
otherwise if you have multiple wostatusage notifications they all appear the same
Also it shows the email address even when it's set to in-app delivery which looks concerning and should not happen
this one might be tricky to get right, if can't then remove it entirely and just show if email or inapp delivery only not the address

View File

@@ -494,18 +494,12 @@ export default {
//clear any errors vm might be around from previous submit
window.$gz.form.deleteAllErrorBoxErrors(vm);
//add in password and login if changed
let submitObject = vm.obj;
if (vm.password != null && vm.password != "") {
submitObject.password = vm.password;
//don't send the deliveryaddress if it's an in-app notification
if (vm.obj.deliveryMethod == 1) {
vm.obj.deliveryAddress = null;
}
if (vm.login != null && vm.login != "") {
submitObject.login = vm.login;
}
if (submitObject.roles == null) {
submitObject.roles = 0;
}
let res = await window.$gz.api.upsert(url, submitObject);
let res = await window.$gz.api.upsert(url, vm.obj);
if (res.error) {
vm.formState.serverError = res.error;
window.$gz.form.setErrorBoxErrors(vm);