Merge pull request #287 from BrainWise-DEV/fix/pos-j-bugs

fix: improve customer dialog behavior by adding nextTick and removing unnecessary console log

Author: MostafaKadry

POS/src/components/sale/CreateCustomerDialog.vue

@@ -385,7 +385,6 @@ const sellingSettingsResource = createResource({
 	auto: false,
 	onError: (err) => log.error("Error loading Selling Settings", err),
 })
-console.log(sellingSettingsResource);
 
 function pickDefault(settingsValue, list, fallbackFn = null) {
 	if (settingsValue && list.includes(settingsValue)) return settingsValue
@@ -575,19 +574,15 @@ watch(showCountryDropdown, async (isOpen) => {
 watch(
 	() => props.modelValue,
 	async (isOpen) => {
-		show.value = isOpen
 		isOpen ? await loadDialogData() : resetForm()
 	}
 )
 
-watch(show, (val) => emit("update:modelValue", val))
-
 // =============================================================================
 // Lifecycle Hooks
 // =============================================================================
 
 onMounted(() => {
-	loadDialogData()
 	document.addEventListener("click", handleClickOutside)
 })
 

POS/src/components/sale/CustomerDialog.vue

@@ -183,7 +183,7 @@
 import { useCustomerSearchStore } from "@/stores/customerSearch"
 import { Button, Dialog } from "frappe-ui"
 import { storeToRefs } from "pinia"
-import { computed, onMounted, ref, watch } from "vue"
+import { computed, nextTick, onMounted, ref, watch } from "vue"
 import CreateCustomerDialog from "./CreateCustomerDialog.vue"
 
 const props = defineProps({
@@ -277,7 +277,9 @@ function selectCustomer(customer) {
 	show.value = false
 }
 
-function createNewCustomer() {
+async function createNewCustomer() {
+	show.value = false
+	await nextTick()
 	showCreateDialog.value = true
 }