Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix db handler #14

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Fix db handler #14

wants to merge 6 commits into from

Conversation

ghostleek
Copy link
Contributor

No description provided.

Assistant generated file changes:
- .replit: Update deployment for demo subdomain
- .streamlit/config.toml: Remove base URL path since using subdomain

---

User prompt:

i want this site to only show on demo.chergpt.sg and use chergpt.sg as another landing page. how do i do this?
Assistant generated file changes:
- .replit: Update custom domain to beta.chergpt.sg

---

User prompt:

instead of demo, call it beta.chergpt.sg
Assistant generated file changes:
- database.py: Fix error handling in save_feedback

---

User prompt:

when generate qns is pressed, nothing happens

UnboundLocalError: cannot access local variable 'cur' where it is not associated with a value
Traceback:
File "/home/runner/03b7bb86-8a74-463e-966e-bafe8cfb3d67/.pythonlibs/lib/python3.12/site-packages/streamlit/runtime/scriptrunner/exec_code.py", line 88, in exec_func_with_error_handling
    result = func()
             ^^^^^^
File "/home/runner/03b7bb86-8a74-463e-966e-bafe8cfb3d67/.pythonlibs/lib/python3.12/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 579, in code_to_exec
    exec(code, module.__dict__)
File "/home/runner/03b7bb86-8a74-463e-966e-bafe8cfb3d67/main.py", line 136, in <module>
    main()
File "/home/runner/03b7bb86-8a74-463e-966e-bafe8cfb3d67/main.py", line 100, in main
    save_feedback(st.session_state.session_uuid, st.session_state.teacher_input)
File "/home/runner/03b7bb86-8a74-463e-966e-bafe8cfb3d67/database.py", line 52, in save_feedback
    cur.close()
    ^^^
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant