CaWittMN08/custom_ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactored onsite meeting to bids all over the codebase.

Browse Source
This commit is contained in:
rocketdebris 2025-12-10 10:25:37 -05:00
parent 6e2535763f
commit 080d9ff1b4
6 changed files with 48 additions and 49 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
custom_ui/api/db/onsite_meetings.py → custom_ui/api/db/bid_meetings.py
View File

@ -3,7 +3,7 @@ import json
from custom_ui.db_utils import build_error_response, build_success_response, process_filters, process_sorting
@frappe.whitelist()
def get_week_onsite_meetings(week_start, week_end):
def get_week_bid_meetings(week_start, week_end):
"""Get On-Site Meetings scheduled within a specific week."""
try:
meetings = frappe.db.get_all(
@ -24,13 +24,13 @@ def get_week_onsite_meetings(week_start, week_end):
return build_error_response(str(e), 500)
@frappe.whitelist()
def get_onsite_meetings(fields=["*"], filters={}):
def get_bid_meetings(fields=["*"], filters={}):
"""Get paginated On-Site Meetings with filtering and sorting support."""
try:
print("DEBUG: Raw onsite meeting options received:", filters)
print("DEBUG: Raw bid meeting options received:", filters)
processed_filters = process_filters(filters)
meetings = frappe.db.get_all(
"On-Site Meeting",
fields=fields,
@ -40,17 +40,17 @@ def get_onsite_meetings(fields=["*"], filters={}):
for meeting in meetings:
address_doc = frappe.get_doc("Address", meeting["address"])
meeting["address"] = address_doc.as_dict()
return build_success_response(
meetings
)
except Exception as e:
frappe.log_error(message=str(e), title="Get On-Site Meetings Failed")
return build_error_response(str(e), 500)
@frappe.whitelist()
def get_unscheduled_onsite_meetings():
def get_unscheduled_bid_meetings():
"""Get On-Site Meetings that are unscheduled."""
try:
meetings = frappe.db.get_all(
@ -63,26 +63,26 @@ def get_unscheduled_onsite_meetings():
except Exception as e:
frappe.log_error(message=str(e), title="Get Unscheduled On-Site Meetings Failed")
return build_error_response(str(e), 500)
@frappe.whitelist()
def create_onsite_meeting(address, notes=""):
def create_bid_meeting(address, notes=""):
"""Create a new On-Site Meeting with Unscheduled status."""
try:
print(f"DEBUG: Creating meeting with address='{address}', notes='{notes}'")
# Validate address parameter
if not address or address == "None" or not address.strip():
return build_error_response("Address is required and cannot be empty.", 400)
# Get the address document name from the full address string
address_name = frappe.db.get_value("Address", filters={"full_address": address}, fieldname="name")
print(f"DEBUG: Address lookup result: address_name='{address_name}'")
if not address_name:
return build_error_response(f"Address '{address}' not found in the system.", 404)
# Create the meeting with Unscheduled status
meeting = frappe.get_doc({
"doctype": "On-Site Meeting",
@ -93,19 +93,19 @@ def create_onsite_meeting(address, notes=""):
meeting.flags.ignore_permissions = True
meeting.insert(ignore_permissions=True)
frappe.db.commit()
# Clear any auto-generated messages from Frappe
frappe.local.message_log = []
print(f"DEBUG: Meeting created successfully: {meeting.name}")
return build_success_response(meeting.as_dict())
except Exception as e:
frappe.log_error(message=str(e), title="Create On-Site Meeting Failed")
return build_error_response(str(e), 500)
@frappe.whitelist()
def update_onsite_meeting(name, data):
def update_bid_meeting(name, data):
"""Update an existing On-Site Meeting."""
defualts = {
"address": None,
@ -141,4 +141,4 @@ def update_onsite_meeting(name, data):
return build_error_response(f"On-Site Meeting '{name}' does not exist.", 404)
except Exception as e:
return build_error_response(str(e), 500)

20
frontend/src/api.js
View File

@ -20,8 +20,8 @@ const FRAPPE_UPSERT_INVOICE_METHOD = "custom_ui.api.db.invoices.upsert_invoice";
const FRAPPE_GET_WARRANTY_CLAIMS_METHOD = "custom_ui.api.db.warranties.get_warranty_claims";
// On-Site Meeting methods
const FRAPPE_GET_WEEK_ONSITE_MEETINGS_METHOD =
"custom_ui.api.db.onsite_meetings.get_week_onsite_meetings";
const FRAPPE_GET_ONSITE_MEETINGS_METHOD = "custom_ui.api.db.onsite_meetings.get_onsite_meetings";
"custom_ui.api.db.bid_meetings.get_week_bid_meetings";
const FRAPPE_GET_ONSITE_MEETINGS_METHOD = "custom_ui.api.db.bid_meetings.get_bid_meetings";
// Address methods
const FRAPPE_GET_ADDRESSES_METHOD = "custom_ui.api.db.addresses.get_addresses";
// Client methods
@ -113,29 +113,29 @@ class Api {
// ON-SITE MEETING METHODS
// ============================================================================
static async getUnscheduledOnSiteMeetings() {
static async getUnscheduledBidMeetings() {
return await this.request(
"custom_ui.api.db.onsite_meetings.get_unscheduled_onsite_meetings",
"custom_ui.api.db.bid_meetings.get_unscheduled_bid_meetings",
);
}
static async getScheduledOnSiteMeetings(fields = ["*"], filters = {}) {
static async getScheduledBidMeetings(fields = ["*"], filters = {}) {
return await this.request(FRAPPE_GET_ONSITE_MEETINGS_METHOD, { fields, filters });
}
static async getWeekOnSiteMeetings(weekStart, weekEnd) {
static async getWeekBidMeetings(weekStart, weekEnd) {
return await this.request(FRAPPE_GET_WEEK_ONSITE_MEETINGS_METHOD, { weekStart, weekEnd });
}
static async updateOnSiteMeeting(name, data) {
return await this.request("custom_ui.api.db.onsite_meetings.update_onsite_meeting", {
static async updateBidMeeting(name, data) {
return await this.request("custom_ui.api.db.bid_meetings.update_bid_meeting", {
name,
data,
});
}
static async createOnSiteMeeting(address, notes = "") {
return await this.request("custom_ui.api.db.onsite_meetings.create_onsite_meeting", {
static async createBidMeeting(address, notes = "") {
return await this.request("custom_ui.api.db.bid_meetings.create_bid_meeting", {
address,
notes,
});

7
frontend/src/components/SideBar.vue
View File

@ -54,9 +54,9 @@ const createButtons = ref([
},
},
{
label: "On-Site Meeting",
label: "Bid",
command: () => {
router.push("/schedule-onsite?new=true");
router.push("/schedule-bid?new=true");
},
},
{
@ -102,10 +102,9 @@ const categories = ref([
{
name: "Clients",
icon: Community,
//url: "/clients"
buttons: clientButtons,
},
{ name: "Bids", icon: Neighbourhood, url: "/schedule-onsite" },
{ name: "Bids", icon: Neighbourhood, url: "/schedule-bid" },
{ name: "Estimates", icon: Calculator, url: "/estimates" },
{ name: "Jobs", icon: Hammer, url: "/jobs" },
{ name: "Payments/Invoices", icon: ReceiveDollars, url: "/invoices" },

2
frontend/src/components/modals/OnSiteMeetingModal.vue → frontend/src/components/modals/BidMeetingModal.vue
View File

@ -6,7 +6,7 @@
@confirm="handleConfirm"
@cancel="handleCancel"
>
<template #title>Schedule New On-Site Meeting</template>
<template #title>Schedule New Bid Meeting</template>
<div class="new-meeting-form">
<div class="form-group">
<label for="meeting-address">Address: <span class="required">*</span></label>

22
frontend/src/components/pages/ScheduleOnSite.vue → frontend/src/components/pages/ScheduleBid.vue
View File

@ -1,8 +1,8 @@
<template>
<div class="schedule-onsite-container">
<div class="schedule-bid-container">
<!-- Header -->
<div class="header">
<h2>On-Site Meeting Schedule</h2>
<h2>Bid Schedule</h2>
<div class="header-controls">
<v-btn
@click="previousWeek"
@ -202,7 +202,7 @@
/>
<!-- New Meeting Modal -->
<OnSiteMeetingModal
<BidMeetingModal
v-model:visible="showNewMeetingModal"
:initial-address="queryAddress"
@confirm="handleNewMeetingConfirm"
@ -214,7 +214,7 @@
<script setup>
import { ref, computed, onMounted, watch } from "vue";
import { useRoute, useRouter } from "vue-router";
import OnSiteMeetingModal from "../modals/OnSiteMeetingModal.vue";
import BidMeetingModal from "../modals/BidMeetingModal.vue";
import MeetingDetailsModal from "../modals/MeetingDetailsModal.vue";
import { useLoadingStore } from "../../stores/loading";
import { useNotificationStore } from "../../stores/notifications-primevue";
@ -470,7 +470,7 @@ const handleNewMeetingConfirm = async (meetingData) => {
loadingStore.setLoading(true);
// Create the meeting via API
const result = await Api.createOnSiteMeeting(meetingData.address, meetingData.notes || "");
const result = await Api.createBidMeeting(meetingData.address, meetingData.notes || "");
showNewMeetingModal.value = false;
@ -502,7 +502,7 @@ const handleNewMeetingCancel = () => {
showNewMeetingModal.value = false;
// Clear query params after canceling
router.push({
path: "/schedule-onsite",
path: "/schedule-bid",
query: {
date: formatDateForUrl(currentWeekStart.value),
},
@ -665,7 +665,7 @@ const handleDrop = async (event, date, time) => {
status: "Scheduled",
};
await Api.updateOnSiteMeeting(droppedMeeting.id, updateData);
await Api.updateBidMeeting(droppedMeeting.id, updateData);
// If this was a reschedule, remove the old meeting from its original position
if (droppedMeeting.isRescheduling && originalMeeting) {
@ -755,7 +755,7 @@ const handleDropToUnscheduled = async (event) => {
try {
loadingStore.setLoading(true);
await Api.updateOnSiteMeeting(droppedMeeting.id, {
await Api.updateBidMeeting(droppedMeeting.id, {
status: "Unscheduled",
start_time: null,
end_time: null,
@ -801,7 +801,7 @@ const handleDropToUnscheduled = async (event) => {
const loadUnscheduledMeetings = async () => {
loadingStore.setLoading(true);
try {
const result = await Api.getUnscheduledOnSiteMeetings();
const result = await Api.getUnscheduledBidMeetings();
// Ensure we always have an array
unscheduledMeetings.value = Array.isArray(result) ? result : [];
console.log("Loaded unscheduled meetings:", unscheduledMeetings.value);
@ -850,7 +850,7 @@ const loadWeekMeetings = async () => {
// Try to get meetings from API
try {
const apiResult = await Api.getWeekOnSiteMeetings(weekStartStr, weekEndStr);
const apiResult = await Api.getWeekBidMeetings(weekStartStr, weekEndStr);
if (Array.isArray(apiResult)) {
// Transform the API data to match what the calendar expects
meetings.value = apiResult
@ -989,7 +989,7 @@ watch(
</script>
<style scoped>
.schedule-onsite-container {
.schedule-bid-container {
padding: 20px;
height: 100vh;
display: flex;

4
frontend/src/router.js
View File

@ -13,7 +13,7 @@ import Home from "./components/pages/Home.vue";
import TestDateForm from "./components/pages/TestDateForm.vue";
import Client from "./components/pages/Client.vue";
import ErrorHandlingDemo from "./components/pages/ErrorHandlingDemo.vue";
import ScheduleOnSite from "./components/pages/ScheduleOnSite.vue";
import ScheduleBid from "./components/pages/ScheduleBid.vue";
import Estimate from "./components/pages/Estimate.vue";
const routes = [
@ -24,7 +24,7 @@ const routes = [
{ path: "/calendar", component: Calendar },
{ path: "/clients", component: Clients },
{ path: "/client", component: Client },
{ path: "/schedule-onsite", component: ScheduleOnSite },
{ path: "/schedule-bid", component: ScheduleBid },
{ path: "/jobs", component: Jobs },
{ path: "/invoices", component: Invoices },
{ path: "/estimates", component: Estimates },