Update app.py

app.py

@@ -3,6 +3,7 @@ import pandas as pd
 from datetime import datetime
 import os
 import sys
+import random
 
 # Disable Streamlit's telemetry and usage statistics
 os.environ['STREAMLIT_ANALYTICS_ENABLED'] = 'false'
@@ -17,34 +18,35 @@ st.set_page_config(
     layout="wide"
 )
 
-# Apply custom CSS for a more professional broadcast look
+# Apply custom CSS for a backlog-style board
 st.markdown("""
 <style>
-    /* Main theme colors for broadcast industry feel */
+    /* Main theme colors */
     :root {
         --primary-color: #0A2647;
         --secondary-color: #144272;
-        --accent-color: #205295;
-        --highlight-color: #2C74B3;
-        --text-color: #0A2647;
-        --card-bg-todo: #f0f7ff;
-        --card-bg-progress: #fff7e6;
-        --card-bg-done: #efffef;
+        --text-color: #333;
+        --todo-color: #03A9F4;
+        --progress-color: #FF5722;
+        --done-color: #4CAF50;
+        --border-color: #e0e0e0;
+        --backlog-color: #8BC34A;
+        --card-bg: #ffffff;
     }
     
     /* Header styling */
     .main-header {
-        background: linear-gradient(90deg, var(--primary-color), var(--accent-color));
+        background: linear-gradient(90deg, var(--primary-color), var(--secondary-color));
         color: white;
         padding: 1rem 2rem;
         border-radius: 5px;
-        margin-bottom: 2rem;
+        margin-bottom: 1.5rem;
         text-align: center;
     }
     
     .main-header h1 {
         margin: 0;
-        font-size: 2.5rem;
+        font-size: 2.2rem;
     }
     
     .company-name {
@@ -63,84 +65,70 @@ st.markdown("""
     }
     
     .task-card {
-        flex: 0 0 300px;
-        background-color: white;
-        border-radius: 12px;
-        padding: 15px;
-        box-shadow: 0 4px 8px rgba(0,0,0,0.08);
-        transition: transform 0.2s, box-shadow 0.2s;
-        position: relative;
+        flex: 0 0 180px;
+        border: 1px solid var(--border-color);
+        border-radius: 4px;
         overflow: hidden;
+        box-shadow: 0 2px 5px rgba(0,0,0,0.1);
+        display: flex;
+        flex-direction: column;
+        background-color: var(--card-bg);
+        height: 150px;
+        transition: transform 0.2s, box-shadow 0.2s;
     }
     
     .task-card:hover {
-        transform: translateY(-5px);
-        box-shadow: 0 6px 12px rgba(0,0,0,0.12);
-    }
-    
-    .task-card.todo {
-        border-top: 5px solid #3498db;
-        background-color: var(--card-bg-todo);
-    }
-    
-    .task-card.in-progress {
-        border-top: 5px solid #f39c12;
-        background-color: var(--card-bg-progress);
-    }
-    
-    .task-card.done {
-        border-top: 5px solid #2ecc71;
-        background-color: var(--card-bg-done);
-    }
-    
-    .task-title {
-        font-size: 1.2rem;
-        font-weight: 600;
-        color: var(--text-color);
-        margin-bottom: 10px;
+        transform: translateY(-3px);
+        box-shadow: 0 4px 8px rgba(0,0,0,0.15);
     }
     
-    .task-meta {
+    .card-header {
+        padding: 8px;
+        color: white;
+        font-weight: bold;
         display: flex;
         justify-content: space-between;
-        color: #555;
-        font-size: 0.9rem;
-        margin-bottom: 12px;
+        align-items: center;
     }
     
-    .task-desc {
-        color: #444;
-        font-size: 0.95rem;
-        margin-bottom: 15px;
+    .card-header.todo {
+        background-color: var(--todo-color);
     }
     
-    .task-footer {
-        display: flex;
-        justify-content: space-between;
-        align-items: center;
-        margin-top: 10px;
+    .card-header.in-progress {
+        background-color: var(--progress-color);
     }
     
-    .status-pill {
-        padding: 4px 10px;
-        border-radius: 15px;
-        font-size: 0.8rem;
-        font-weight: 500;
+    .card-header.done {
+        background-color: var(--done-color);
     }
     
-    .status-pill.todo {
-        background-color: #e1f0ff;
-        color: #3498db;
+    .card-header.backlog {
+        background-color: var(--backlog-color);
     }
     
-    .status-pill.in-progress {
-        background-color: #fff4e0;
-        color: #f39c12;
+    .task-id {
+        font-size: 14px;
+        font-weight: bold;
     }
     
-    .status-pill.done {
-        background-color: #e0ffe0;
-        color: #2ecc71;
+    .task-title {
+        padding: 10px;
+        font-weight: bold;
+        color: var(--text-color);
+        font-size: 14px;
+        height: 40px;
+        overflow: hidden;
+        text-overflow: ellipsis;
+    }
+    
+    .task-assignee {
+        padding: 0 10px;
+        color: #666;
+        font-size: 12px;
+        font-style: italic;
+        margin-top: auto;
+        margin-bottom: 10px;
     }
     
     /* Form styling */
@@ -156,7 +144,7 @@ st.markdown("""
     /* Button styling */
     .stButton > button {
         width: 100%;
-        background-color: var(--accent-color);
+        background-color: var(--secondary-color);
         color: white;
         border: none;
         padding: 10px 15px;
@@ -165,7 +153,7 @@ st.markdown("""
     }
     
     .stButton > button:hover {
-        background-color: var(--highlight-color);
+        background-color: var(--primary-color);
     }
     
     /* Hide storage path message */
@@ -184,37 +172,22 @@ st.markdown("""
 </style>
 """, unsafe_allow_html=True)
 
-# Color mapping
+# Status color mapping
 STATUS_COLORS = {
     "To Do": "todo",
     "In Progress": "in-progress",
-    "Done": "done"
+    "Done": "done",
+    "Backlog": "backlog"
 }
 
 # Set up paths
 TASKS_FILE = "tasks.csv"  # Simplified path for tasks
-
-# Check all possible locations for Assignee.txt
-def find_assignee_file():
-    possible_locations = [
-        "Assignee.txt",                    # Root directory
-        "./Assignee.txt",                  # Explicit current directory
-        "/app/Assignee.txt",               # Docker app directory
-        os.path.join(os.getcwd(), "Assignee.txt"),  # Full path to current directory
-        "/data/Assignee.txt",              # Data directory
-        "../Assignee.txt",                 # Parent directory
-    ]
-    
-    for location in possible_locations:
-        if os.path.exists(location):
-            return location
-    
-    return None
+ASSIGNEE_FILE = "Assignee.txt"  # Direct path to Assignee.txt
 
 def create_empty_df():
     """Create an empty DataFrame with the required columns"""
     return pd.DataFrame(columns=[
-        'Title', 'Description', 'Assignee', 'Status', 'Due Date'
+        'ID', 'Title', 'Description', 'Assignee', 'Status', 'Due Date'
     ])
 
 def load_tasks():
@@ -224,6 +197,9 @@ def load_tasks():
             df = pd.read_csv(TASKS_FILE, parse_dates=['Due Date'])
             if df.empty:
                 return create_empty_df()
+            # Ensure ID column exists
+            if 'ID' not in df.columns:
+                df['ID'] = [f"#{i}" for i in range(1, len(df) + 1)]
             return df
         else:
             return create_empty_df()
@@ -240,51 +216,108 @@ def save_tasks():
         return False
 
 def load_assignees():
-    """Load assignees from text file with extensive debugging and fallbacks"""
-    default_assignees = ["John Doe", "Jane Smith", "Alex Johnson", "Maria Garcia"]
+    """Load assignees from Assignee.txt with strict requirement"""
+    # Always log file contents for debugging
+    log_files_in_directory()
     
-    # Find the Assignee.txt file
-    assignee_file_path = find_assignee_file()
+    # Check direct path first
+    if os.path.exists(ASSIGNEE_FILE):
+        return parse_assignee_file(ASSIGNEE_FILE)
     
-    if not assignee_file_path:
-        # Debugging: log available files in the current directory
-        files_in_cwd = os.listdir(os.getcwd())
-        print(f"Files in current directory: {files_in_cwd}")
-        return default_assignees
+    # If not found, check other common locations
+    possible_locations = [
+        "./Assignee.txt",
+        "/app/Assignee.txt",
+        os.path.join(os.getcwd(), "Assignee.txt"),
+        "/data/Assignee.txt",
+        "../Assignee.txt",
+    ]
     
+    for location in possible_locations:
+        if os.path.exists(location):
+            return parse_assignee_file(location)
+    
+    # If we reach here, the file wasn't found anywhere
+    st.error("Assignee.txt file not found. Please create this file in the app directory.")
+    return ["ERROR: Please create Assignee.txt file"]
+
+def parse_assignee_file(file_path):
+    """Parse the assignee file and extract names"""
     try:
-        assignees = []
-        with open(assignee_file_path, "r") as f:
+        with open(file_path, "r") as f:
             content = f.read()
-            # Debug: Print the content of the file
-            print(f"Content of {assignee_file_path}: {content}")
+            # Debug: Print content to logs
+            print(f"Content of {file_path}:\n{content}")
             
+            # Parse lines and extract names
+            assignees = []
             lines = [line.strip() for line in content.split('\n') if line.strip()]
+            
             for line in lines:
                 if '-' in line:
+                    # Format: "Name - Role"
                     name_part = line.split('-')[0].strip()
-                    if ' ' in name_part:
+                    if name_part:
                         assignees.append(name_part)
-                else:
-                    # If there's no dash, just use the whole line if it has a space
-                    if ' ' in line:
-                        assignees.append(line)
-        
-        # If we successfully parsed assignees, return them
-        if assignees:
+                elif line:
+                    # Just use the line as is if no dash
+                    assignees.append(line)
+            
+            # Log what we found
+            print(f"Found {len(assignees)} assignees: {assignees}")
+            
+            if not assignees:
+                st.warning(f"No valid assignee names found in {file_path}. File exists but may be empty or improperly formatted.")
+                return ["ERROR: Empty or invalid Assignee.txt"]
+                
             return assignees
+    except Exception as e:
+        st.error(f"Error reading assignee file: {str(e)}")
+        print(f"Error reading {file_path}: {str(e)}")
+        return ["ERROR: Cannot read Assignee.txt"]
+
+def log_files_in_directory():
+    """Log files in current directory for debugging"""
+    try:
+        cwd = os.getcwd()
+        files = os.listdir(cwd)
+        print(f"Files in {cwd}: {files}")
         
-        # If no assignees were extracted, return defaults
-        return default_assignees
+        # Also check parent directory
+        parent = os.path.dirname(cwd)
+        if os.path.exists(parent):
+            parent_files = os.listdir(parent)
+            print(f"Files in parent directory {parent}: {parent_files}")
+            
+        # Check app directory if different
+        app_dir = "/app"
+        if os.path.exists(app_dir) and app_dir != cwd:
+            app_files = os.listdir(app_dir)
+            print(f"Files in {app_dir}: {app_files}")
     except Exception as e:
-        print(f"Error loading assignees: {str(e)}")
-        return default_assignees
+        print(f"Error listing directory contents: {str(e)}")
+
+# Generate a new task ID
+def generate_task_id():
+    if len(st.session_state.tasks) == 0:
+        return "#1"
+    
+    # Extract existing IDs and find the maximum
+    try:
+        existing_ids = [int(task_id.replace('#', '')) for task_id in st.session_state.tasks['ID'].tolist() if task_id.startswith('#')]
+        if existing_ids:
+            return f"#{max(existing_ids) + 1}"
+        else:
+            return "#1"
+    except:
+        # If any error occurs, generate a random ID
+        return f"#{random.randint(1, 100)}"
 
 # Initialize session state ONLY ONCE
 if 'tasks' not in st.session_state:
     st.session_state.tasks = load_tasks()
 
-# Load assignees once and reuse
+# Load assignees directly from the file
 assignee_list = load_assignees()
 
 # Custom header
@@ -301,12 +334,14 @@ with st.sidebar:
     title = st.text_input("Task Title")
     description = st.text_area("Description")
     assignee = st.selectbox("Assignee", assignee_list)
-    status = st.selectbox("Status", ["To Do", "In Progress", "Done"])
+    status = st.selectbox("Status", ["Backlog", "To Do", "In Progress", "Done"])
     due_date = st.date_input("Due Date")
     
     if st.button("Add Task"):
         if title:  # Basic validation
+            task_id = generate_task_id()
             new_task = {
+                'ID': task_id,
                 'Title': title,
                 'Description': description,
                 'Assignee': assignee,
@@ -323,52 +358,48 @@ with st.sidebar:
         else:
             st.warning("Please enter a task title")
 
-# Display tasks in a horizontal grid
+# Display tasks in a backlog-style grid
 if st.session_state.tasks.empty:
     st.info("No tasks yet. Add your first task using the sidebar.")
 else:
     # Start the grid container
     st.markdown('<div class="task-grid">', unsafe_allow_html=True)
     
-    # Render each task as a card in the grid
+    # Render each task as a backlog item card
     for idx, task in st.session_state.tasks.iterrows():
-        status_class = STATUS_COLORS.get(task['Status'], "todo")
+        status_class = STATUS_COLORS.get(task['Status'], "backlog")
+        task_id = task['ID'] if 'ID' in task and pd.notna(task['ID']) else f"#{idx+1}"
         
-        # Format the date
-        due_date = task['Due Date']
-        if isinstance(due_date, datetime):
-            formatted_date = due_date.strftime('%Y-%m-%d')
-        else:
-            formatted_date = due_date
-            
         # Create the card HTML
         card_html = f"""
-        <div class="task-card {status_class}">
-            <div class="task-title">{task['Title']}</div>
-            <div class="task-meta">
-                <span><strong>Assignee:</strong> {task['Assignee']}</span>
-                <span><strong>Due:</strong> {formatted_date}</span>
-            </div>
-            <div class="task-desc">{task['Description']}</div>
-            <div class="task-footer">
-                <span class="status-pill {status_class}">{task['Status']}</span>
+        <div class="task-card">
+            <div class="card-header {status_class}">
+                <span class="task-id">{task_id}</span>
+                <span>{task['Status']}</span>
             </div>
+            <div class="task-title">{task['Title']}</div>
+            <div class="task-assignee">{task['Assignee']}</div>
         </div>
         """
         
         # Render the card
         st.markdown(card_html, unsafe_allow_html=True)
         
-        # Hidden status update below each card
-        # We'll use columns to make this less prominent
-        cols = st.columns([3, 1])
-        with cols[1]:
+        # Add a hidden status update below each card
+        # Create an invisible element for status updates
+        with st.container():
+            st.markdown(f"""
+            <div style="display: none;">
+                <p>Update {task_id}</p>
+            </div>
+            """, unsafe_allow_html=True)
+            
             new_status = st.selectbox(
-                "Update Status",  # Provide a label but hide it
-                ["To Do", "In Progress", "Done"],
-                index=["To Do", "In Progress", "Done"].index(task['Status']),
+                f"Update {task_id}",
+                ["Backlog", "To Do", "In Progress", "Done"],
+                index=["Backlog", "To Do", "In Progress", "Done"].index(task['Status']) if task['Status'] in ["Backlog", "To Do", "In Progress", "Done"] else 0,
                 key=f"status_{idx}",
-                label_visibility="hidden"  # Use hidden instead of collapsed
+                label_visibility="hidden"
             )
             
             if new_status != task['Status']: