Update app.py

app.py

@@ -1,17 +1,17 @@
-import gradio as gr
 import pandas as pd
 import gspread
-from oauth2client.service_account import ServiceAccountCredentials
+import gradio as gr
 import plotly.express as px
+from oauth2client.service_account import ServiceAccountCredentials
 
-# ------------------ Authentication ------------------
+# ------------------ AUTH ------------------
 VALID_USERS = {
     "[email protected]": "Pass.123",
     "[email protected]": "Pass.123",
     "[email protected]": "Pass.123"
 }
 
-# ------------------ Data Setup ------------------
+# ------------------ GOOGLE SHEET SETUP ------------------
 scope = ["https://spreadsheets.google.com/feeds", "https://www.googleapis.com/auth/drive"]
 creds = ServiceAccountCredentials.from_json_keyfile_name("tough-star.json", scope)
 client = gspread.authorize(creds)
@@ -20,10 +20,10 @@ sheet = client.open_by_url(sheet_url).worksheet("Calls")
 data = sheet.get_all_records()
 df = pd.DataFrame(data)
 
+# ------------------ DATA CLEANING ------------------
 df['Timestamp'] = pd.to_datetime(df['Timestamp'], dayfirst=True, errors='coerce')
 df['Date'] = df['Timestamp'].dt.date.astype(str)
 df['Time'] = df['Timestamp'].dt.time
-
 location_split = df['Location'].str.split(',', expand=True)
 df['Latitude'] = pd.to_numeric(location_split[0], errors='coerce')
 df['Longitude'] = pd.to_numeric(location_split[1], errors='coerce')
@@ -32,10 +32,9 @@ df = df[(df['Latitude'] != 0) & (df['Longitude'] != 0)]
 df = df.sort_values(by=['Rep Name', 'Timestamp'])
 df['Time Diff (min)'] = df.groupby(['Rep Name', 'Date'])['Timestamp'].diff().dt.total_seconds().div(60).fillna(0)
 df['Visit Order'] = df.groupby(['Rep Name', 'Date']).cumcount() + 1
-
 ALL_REPS = sorted(df['Rep Name'].dropna().unique())
 
-# ------------------ App Logic ------------------
+# ------------------ DASHBOARD FUNCTIONS ------------------
 def generate_summary(date_str):
     day_df = df[df['Date'] == date_str]
     active = day_df.groupby('Rep Name').size().reset_index(name='Total Visits')
@@ -90,8 +89,8 @@ def show_map(date_str, rep):
 
     fig.update_layout(mapbox_style="open-street-map", title=f"📍 {rep}'s Route on {date_str}")
 
-    table = subset[[
-        'Visit Order', 'Dealership Name', 'Time', 'Time Diff (min)',
+    table = subset[[ 
+        'Visit Order', 'Dealership Name', 'Time', 'Time Diff (min)', 
         'Type of call', 'Sales or service'
     ]].rename(columns={
         'Dealership Name': '🧭 Dealer',
@@ -113,13 +112,11 @@ def show_map(date_str, rep):
     table = pd.concat([table, summary_row], ignore_index=True)
     return table, fig
 
-# ------------------ Login + Protected UI ------------------
+# ------------------ GRADIO APP ------------------
 with gr.Blocks() as app:
-    state = gr.State(False)
-
     with gr.Row():
-        with gr.Column(scale=1):
-            login_title = gr.Markdown("## 🔐 Login Required")
+        with gr.Column(visible=True) as login_ui:
+            gr.Markdown("## 🔐 Login Required")
             email = gr.Textbox(label="Email")
             password = gr.Textbox(label="Password", type="password")
             login_btn = gr.Button("Login")
@@ -146,12 +143,12 @@ with gr.Blocks() as app:
                 date_picker.change(fn=get_reps, inputs=date_picker, outputs=rep_picker)
                 btn.click(fn=show_map, inputs=[date_picker, rep_picker], outputs=[table, map_plot])
 
-    def check_login(u, p):
-        if VALID_USERS.get(u) == p:
+    def do_login(user, pw):
+        if VALID_USERS.get(user) == pw:
             return gr.update(visible=False), gr.update(visible=True), ""
         else:
-            return gr.update(), gr.update(), "❌ Incorrect email or password."
+            return gr.update(visible=True), gr.update(visible=False), "❌ Invalid email or password."
 
-    login_btn.click(fn=check_login, inputs=[email, password], outputs=[login_title, main_ui, login_msg])
+    login_btn.click(fn=do_login, inputs=[email, password], outputs=[login_ui, main_ui, login_msg])
 
 app.launch()