Push to week 4

src/views/week4.py

@@ -103,13 +103,6 @@ def show_week4_content():
             - Make sure you can present everything within your time limit
             """)
             
-            presentation_notes = st.text_area(
-                "Any notes about your full case presentation (optional):",
-                placeholder="e.g., 'I need to work on my pacing' or 'The transition between my first and second argument needs work'",
-                height=100,
-                key="full_case_notes"
-            )
-            
         elif presentation_option == "2. Present 1 argument you think is the best and explain why":
             st.markdown("""
             **Option 2: Best Argument Focus**
@@ -122,13 +115,6 @@ def show_week4_content():
             - Connect it to your overall case strategy
             """)
             
-            best_argument = st.text_area(
-                "Which argument from your case do you think is the best? Explain why:",
-                placeholder="e.g., 'My economic impact argument is strongest because...' or 'The moral obligation argument works best because...'",
-                height=150,
-                key="best_argument_explanation"
-            )
-            
         elif presentation_option == "3. Talk about what you think is a good strategy":
             st.markdown("""
             **Option 3: Strategy Discussion**
@@ -140,13 +126,6 @@ def show_week4_content():
             - Discuss how your arguments complement each other
             - Share insights about effective debate strategy
             """)
-            
-            strategy_discussion = st.text_area(
-                "Discuss your debate strategy and approach:",
-                placeholder="e.g., 'My strategy focuses on...' or 'I think the most effective approach is...' or 'My arguments work together by...'",
-                height=150,
-                key="strategy_discussion"
-            )
         
         st.markdown("---")
         st.markdown("### **🎯 Ready to Practice!**")
@@ -362,9 +341,9 @@ def show_week4_content():
             
             st.markdown("### **📄 Your Debate Case Document**")
             
-            # Convert Google Doc URL to embed URL
+            # Convert Google Doc URL to embed URL (allow editing)
             google_doc_url = student_case['google_doc_link']
-            embed_url = google_doc_url.replace("edit", "preview")
+            embed_url = google_doc_url  # Keep as edit mode to allow editing
             
             # Display the Google Doc in an iframe
             st.markdown(f"""
@@ -378,8 +357,7 @@ def show_week4_content():
             """, unsafe_allow_html=True)
             
             # Add download button
-            download_url = google_doc_url.replace("edit", "export?format=docx")
-            st.markdown(f"[📥 Download as Word Document]({download_url})")
+            st.markdown(f"[📥 Download as Word Document]({embed_url})")
             
             if student_case.get('description'):
                 st.markdown(f"**Description:** {student_case['description']}")

tests/add_debate_case.py

@@ -0,0 +1,81 @@
+import streamlit as st
+import sys
+import os
+sys.path.append(os.path.join(os.path.dirname(__file__), 'src'))
+
+from utils import init_supabase
+
+def add_debate_case():
+    st.title("📝 Add Debate Case")
+    
+    # Initialize Supabase
+    supabase = init_supabase()
+    if not supabase:
+        st.error("❌ Cannot connect to database")
+        return
+    
+    st.markdown("---")
+    
+    # Form to add debate case
+    with st.form("add_debate_case_form"):
+        st.subheader("Add New Debate Case")
+        
+        # Student ID input
+        student_id = st.text_input("Student ID (username):", placeholder="e.g., lily, reed")
+        
+        # Google Doc URL input
+        google_doc_url = st.text_input("Google Doc URL:", 
+                                     value="https://docs.google.com/document/d/1J8Unkklwhk778jcARshISK0AdBZ34XikQpeBDAWAxY0/edit",
+                                     placeholder="https://docs.google.com/document/d/...")
+        
+        # Description input
+        description = st.text_area("Description:", placeholder="Brief description of the debate case")
+        
+        # Submit button
+        submitted = st.form_submit_button("Add Debate Case")
+        
+        if submitted:
+            if student_id and google_doc_url:
+                try:
+                    # Insert the debate case
+                    data = {
+                        'student_id': student_id,
+                        'google_doc_link': google_doc_url,
+                        'description': description
+                    }
+                    
+                    response = supabase.table('student_debate_case').insert(data).execute()
+                    
+                    if response.data:
+                        st.success(f"✅ Successfully added debate case for {student_id}")
+                        st.write("**Added data:**")
+                        st.json(response.data[0])
+                    else:
+                        st.error("❌ Failed to add debate case")
+                        
+                except Exception as e:
+                    st.error(f"❌ Error adding debate case: {str(e)}")
+            else:
+                st.warning("⚠️ Please fill in Student ID and Google Doc URL")
+    
+    st.markdown("---")
+    
+    # Show existing debate cases
+    st.subheader("Current Debate Cases")
+    try:
+        response = supabase.table('student_debate_case').select('*').execute()
+        if response.data:
+            st.success(f"✅ Found {len(response.data)} debate cases:")
+            for i, case in enumerate(response.data, 1):
+                st.write(f"**{i}.** Student: `{case.get('student_id', 'N/A')}`")
+                st.write(f"    Link: {case.get('google_doc_link', 'N/A')}")
+                st.write(f"    Description: {case.get('description', 'N/A')}")
+                st.write(f"    Created: {case.get('created_at', 'N/A')}")
+                st.write("---")
+        else:
+            st.info("📝 No debate cases found in table")
+    except Exception as e:
+        st.error(f"❌ Error retrieving debate cases: {str(e)}")
+
+if __name__ == "__main__":
+    add_debate_case() 

tests/debug_page.py

@@ -0,0 +1,95 @@
+import streamlit as st
+import sys
+import os
+sys.path.append(os.path.join(os.path.dirname(__file__), 'src'))
+
+from utils import init_supabase, get_student_debate_case
+
+def main():
+    st.title("🔍 Week 4 Debate Case Debug")
+    
+    st.markdown("---")
+    
+    # Test 1: Supabase Connection
+    st.subheader("1. Testing Supabase Connection")
+    try:
+        supabase = init_supabase()
+        if supabase:
+            st.success("✅ Supabase connection successful")
+        else:
+            st.error("❌ Supabase connection failed")
+            return
+    except Exception as e:
+        st.error(f"❌ Supabase connection error: {str(e)}")
+        return
+    
+    # Test 2: Check if table exists
+    st.subheader("2. Testing student_debate_case table")
+    try:
+        response = supabase.table('student_debate_case').select('*').limit(1).execute()
+        st.success(f"✅ Table accessible, found {len(response.data)} records")
+    except Exception as e:
+        st.error(f"❌ Table access error: {str(e)}")
+        return
+    
+    # Test 3: List all records
+    st.subheader("3. Listing all debate cases in table")
+    try:
+        response = supabase.table('student_debate_case').select('*').execute()
+        if response.data:
+            st.success(f"✅ Found {len(response.data)} debate cases:")
+            for i, case in enumerate(response.data, 1):
+                st.write(f"   **{i}.** Student ID: `{case.get('student_id', 'N/A')}`")
+                st.write(f"       Link: {case.get('google_doc_link', 'N/A')}")
+                st.write(f"       Created: {case.get('created_at', 'N/A')}")
+                st.write("---")
+        else:
+            st.warning("❌ No debate cases found in table")
+    except Exception as e:
+        st.error(f"❌ Error listing cases: {str(e)}")
+    
+    # Test 4: Test with specific usernames
+    st.subheader("4. Testing with specific usernames")
+    test_usernames = ["lily", "reed", "instructor", "admin"]
+    
+    for username in test_usernames:
+        try:
+            result = get_student_debate_case(username)
+            if result:
+                st.success(f"✅ Found case for '{username}': {result.get('google_doc_link', 'N/A')}")
+            else:
+                st.info(f"❌ No case found for '{username}'")
+        except Exception as e:
+            st.error(f"❌ Error testing '{username}': {str(e)}")
+    
+    # Test 5: Test with current user
+    st.subheader("5. Testing with current user")
+    if st.session_state.get('authenticated', False):
+        username = st.session_state.get('username')
+        st.write(f"**Current user:** {username}")
+        
+        try:
+            result = get_student_debate_case(username)
+            if result:
+                st.success(f"✅ Found case for current user '{username}': {result.get('google_doc_link', 'N/A')}")
+            else:
+                st.warning(f"❌ No case found for current user '{username}'")
+        except Exception as e:
+            st.error(f"❌ Error testing current user: {str(e)}")
+    else:
+        st.warning("⚠️ No user currently authenticated")
+    
+    # Test 6: Check RLS policies
+    st.subheader("6. RLS Policy Check")
+    st.info("""
+    **RLS Policies Applied:**
+    - Users can read their own debate cases
+    - Users can insert their own debate cases  
+    - Users can update their own debate cases
+    - Users can delete their own debate cases
+    
+    **Note:** If you're testing with a service role or admin user, you might need additional policies.
+    """)
+
+if __name__ == "__main__":
+    main() 

tests/debug_week4.py

@@ -0,0 +1,101 @@
+#!/usr/bin/env python3
+"""
+Debug script for Week 4 debate case functionality
+Run this script to test if student debate cases are being retrieved correctly
+"""
+
+import os
+import sys
+sys.path.append(os.path.join(os.path.dirname(__file__), '..', 'src'))
+
+# Import Supabase directly for testing
+from supabase import create_client, Client
+
+def debug_week4_debate_case():
+    """Debug the Week 4 debate case functionality"""
+    print("🔍 Debugging Week 4 Debate Case Functionality")
+    print("=" * 60)
+    
+    # Test 1: Supabase Connection
+    print("1. Testing Supabase Connection...")
+    try:
+        # Get credentials from environment variables
+        url = os.getenv("SUPABASE_URL")
+        key = os.getenv("SUPABASE_SERVICE_ROLE_KEY")  # Use service role key
+        
+        if not url or not key:
+            print("   ❌ Missing Supabase credentials in environment variables")
+            print(f"      SUPABASE_URL: {'Set' if url else 'Missing'}")
+            print(f"      SUPABASE_SERVICE_ROLE_KEY: {'Set' if key else 'Missing'}")
+            print("   💡 Make sure to set SUPABASE_SERVICE_ROLE_KEY environment variable")
+            return
+        
+        # Create Supabase client directly with service role
+        supabase = create_client(url, key)
+        print("   ✅ Supabase connection successful (using service role)")
+        
+    except Exception as e:
+        print(f"   ❌ Supabase connection error: {str(e)}")
+        return
+    
+    # Test 2: Check if table exists
+    print("\n2. Testing student_debate_case table...")
+    try:
+        response = supabase.table('student_debate_case').select('*').limit(1).execute()
+        print(f"   ✅ Table accessible, found {len(response.data)} records")
+    except Exception as e:
+        print(f"   ❌ Table access error: {str(e)}")
+        return
+    
+    # Test 3: List all records
+    print("\n3. Listing all debate cases in table...")
+    try:
+        response = supabase.table('student_debate_case').select('*').execute()
+        if response.data:
+            print(f"   ✅ Found {len(response.data)} debate cases:")
+            for i, case in enumerate(response.data, 1):
+                print(f"      {i}. Student ID: {case.get('student_id', 'N/A')}")
+                print(f"         Google Doc: {case.get('google_doc_link', 'N/A')}")
+                print(f"         Description: {case.get('description', 'N/A')}")
+                print(f"         Created: {case.get('created_at', 'N/A')}")
+                print()
+        else:
+            print("   ❌ No debate cases found in table")
+    except Exception as e:
+        print(f"   ❌ Error listing cases: {str(e)}")
+    
+    # Test 4: Test with specific usernames
+    print("\n4. Testing with specific usernames...")
+    test_usernames = ["test_student", "student1", "demo_user", "admin"]
+    
+    for username in test_usernames:
+        try:
+            # Test the query directly
+            response = supabase.table('student_debate_case').select('*').eq('student_id', username).order('created_at', desc=True).limit(1).execute()
+            if response.data:
+                print(f"   ✅ Found case for '{username}': {response.data[0].get('google_doc_link', 'N/A')}")
+            else:
+                print(f"   ❌ No case found for '{username}'")
+        except Exception as e:
+            print(f"   ❌ Error testing '{username}': {str(e)}")
+    
+    # Test 5: Check environment variables
+    print("\n5. Checking environment variables...")
+    try:
+        url = os.getenv("SUPABASE_URL")
+        key = os.getenv("SUPABASE_KEY")
+        
+        if url:
+            print(f"   ✅ SUPABASE_URL found: {url[:20]}...")
+        else:
+            print("   ❌ SUPABASE_URL not found")
+            
+        if key:
+            print(f"   ✅ SUPABASE_KEY found: {key[:10]}...")
+        else:
+            print("   ❌ SUPABASE_KEY not found")
+    except Exception as e:
+        print(f"   ❌ Error checking environment: {str(e)}")
+
+if __name__ == "__main__":
+    debug_week4_debate_case() 

tests/test_week4_debate_case.py

@@ -0,0 +1,182 @@
+import streamlit as st
+import os
+import sys
+sys.path.append(os.path.join(os.path.dirname(__file__), '..', 'src'))
+
+from utils import init_supabase, get_student_debate_case
+from supabase import create_client, Client
+
+def test_supabase_connection():
+    """Test if Supabase connection is working"""
+    print("=== Testing Supabase Connection ===")
+    
+    try:
+        supabase = init_supabase()
+        if supabase:
+            print("✅ Supabase connection successful")
+            return True
+        else:
+            print("❌ Supabase connection failed")
+            return False
+    except Exception as e:
+        print(f"❌ Supabase connection error: {str(e)}")
+        return False
+
+def test_student_debate_case_table():
+    """Test if student_debate_case table exists and is accessible"""
+    print("\n=== Testing student_debate_case Table ===")
+    
+    try:
+        supabase = init_supabase()
+        if not supabase:
+            print("❌ Cannot test table - no Supabase connection")
+            return False
+        
+        # Try to select from the table
+        response = supabase.table('student_debate_case').select('*').limit(1).execute()
+        print("✅ student_debate_case table is accessible")
+        print(f"   Found {len(response.data)} records in table")
+        return True
+        
+    except Exception as e:
+        print(f"❌ Error accessing student_debate_case table: {str(e)}")
+        return False
+
+def test_get_student_debate_case_function():
+    """Test the get_student_debate_case function with a test username"""
+    print("\n=== Testing get_student_debate_case Function ===")
+    
+    # Test with a sample username
+    test_username = "test_student"
+    
+    try:
+        result = get_student_debate_case(test_username)
+        if result:
+            print(f"✅ Found debate case for {test_username}")
+            print(f"   Google Doc Link: {result.get('google_doc_link', 'N/A')}")
+            print(f"   Description: {result.get('description', 'N/A')}")
+            print(f"   Created: {result.get('created_at', 'N/A')}")
+        else:
+            print(f"❌ No debate case found for {test_username}")
+        
+        return result is not None
+        
+    except Exception as e:
+        print(f"❌ Error in get_student_debate_case: {str(e)}")
+        return False
+
+def test_all_students_in_table():
+    """List all students who have debate cases"""
+    print("\n=== Testing All Students in Table ===")
+    
+    try:
+        supabase = init_supabase()
+        if not supabase:
+            print("❌ Cannot test - no Supabase connection")
+            return False
+        
+        # Get all records from the table
+        response = supabase.table('student_debate_case').select('*').execute()
+        
+        if response.data:
+            print(f"✅ Found {len(response.data)} debate cases in table:")
+            for i, case in enumerate(response.data, 1):
+                print(f"   {i}. Student: {case.get('student_id', 'N/A')}")
+                print(f"      Link: {case.get('google_doc_link', 'N/A')}")
+                print(f"      Created: {case.get('created_at', 'N/A')}")
+                print()
+        else:
+            print("❌ No debate cases found in table")
+        
+        return len(response.data) > 0
+        
+    except Exception as e:
+        print(f"❌ Error listing students: {str(e)}")
+        return False
+
+def test_username_matching():
+    """Test if the username from session state matches any student_id in the table"""
+    print("\n=== Testing Username Matching ===")
+    
+    try:
+        supabase = init_supabase()
+        if not supabase:
+            print("❌ Cannot test - no Supabase connection")
+            return False
+        
+        # Get all student_ids from the table
+        response = supabase.table('student_debate_case').select('student_id').execute()
+        
+        if response.data:
+            student_ids = [case['student_id'] for case in response.data]
+            print(f"✅ Found {len(student_ids)} student IDs in table:")
+            for student_id in student_ids:
+                print(f"   - {student_id}")
+        else:
+            print("❌ No student IDs found in table")
+        
+        return len(response.data) > 0
+        
+    except Exception as e:
+        print(f"❌ Error testing username matching: {str(e)}")
+        return False
+
+def test_google_doc_url_conversion():
+    """Test the Google Doc URL conversion logic"""
+    print("\n=== Testing Google Doc URL Conversion ===")
+    
+    # Test URLs
+    test_urls = [
+        "https://docs.google.com/document/d/1J8Unkklwhk778jcARshISK0AdBZ34XikQpeBDAWAxY0/edit?tab=t.3t21ivxifp52#heading=h.vsxctn9u5gyw",
+        "https://docs.google.com/document/d/test123/edit",
+        "https://docs.google.com/document/d/another-test/edit?usp=sharing"
+    ]
+    
+    for url in test_urls:
+        embed_url = url.replace("edit", "preview")
+        download_url = url.replace("edit", "export?format=docx")
+        
+        print(f"Original: {url}")
+        print(f"Embed:    {embed_url}")
+        print(f"Download: {download_url}")
+        print()
+
+def main():
+    """Run all tests"""
+    print("🔍 Week 4 Debate Case Debug Tests")
+    print("=" * 50)
+    
+    # Run tests
+    tests = [
+        test_supabase_connection,
+        test_student_debate_case_table,
+        test_get_student_debate_case_function,
+        test_all_students_in_table,
+        test_username_matching,
+        test_google_doc_url_conversion
+    ]
+    
+    results = []
+    for test in tests:
+        try:
+            result = test()
+            results.append(result)
+        except Exception as e:
+            print(f"❌ Test {test.__name__} failed with exception: {str(e)}")
+            results.append(False)
+    
+    # Summary
+    print("\n" + "=" * 50)
+    print("📊 Test Summary:")
+    passed = sum(results)
+    total = len(results)
+    print(f"✅ Passed: {passed}/{total}")
+    print(f"❌ Failed: {total - passed}/{total}")
+    
+    if passed == total:
+        print("🎉 All tests passed!")
+    else:
+        print("⚠️ Some tests failed. Check the output above for details.")
+
+if __name__ == "__main__":
+    main()