Feat add db nps #12
Merged
Feat add db nps #12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Assistant generated file changes: - main.py: Fix line length and trailing whitespace, Fix trailing whitespace and exception handling - test_main.py: Fix line length in test file --- User prompt: Run pylint $(git ls-files '*.py') ************* Module main main.py:50:0: C0301: Line too long (111/100) (line-too-long) main.py:52:0: C0301: Line too long (113/100) (line-too-long) main.py:82:0: C0303: Trailing whitespace (trailing-whitespace) main.py:84:0: C0303: Trailing whitespace (trailing-whitespace) main.py:103:0: C0303: Trailing whitespace (trailing-whitespace) main.py:115:0: C0301: Line too long (107/100) (line-too-long) main.py:26:7: W0718: Catching too general exception Exception (broad-exception-caught) ************* Module test_main test_main.py:25:0: C0301: Line too long (134/100) (line-too-long) test_main.py:27:0: C0301: Line too long (107/100) (line-too-long)
Assistant generated file changes: - database.py: Switch to Replit's database --- User prompt: is installing sqlite necessary to use the db function in replit?
Assistant generated file changes: - main.py: Add star rating component --- User prompt: how about using https://github.com/flucas96/streamlit_star_rating instead
Assistant generated file changes: - requirements.txt: Update star rating package name --- User prompt: change the command to pip install st-star-rating
Assistant generated file changes: - main.py: Fix star rating implementation --- User prompt: no nps survey shows. why?
Assistant generated file changes:
- main.py: Fix star rating implementation
---
User prompt:
i think the component isn't imported correctly
It should look like
from streamlit_star_rating import st_star_rating
$ stars = st_star_rating("Please rate you experience", maxValue=5, defaultValue=3, key="rating")
Assistant generated file changes:
- main.py: Fix star rating variable name
---
User prompt:
help resolve error:
2025-01-04 09:52:13.751 Uncaught app execution
Traceback (most recent call last):
File "/home/runner/strExitCard/.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/strExitCard/.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/strExitCard/main.py", line 134, in <module>
main()
File "/home/runner/strExitCard/main.py", line 118, in main
if rating:
^^^^^^
NameError: name 'rating' is not defined
Assistant generated file changes: - main.py: Update star rating and add email field --- User prompt: should make min stars 5 by default. remove the text "please rate your experience" include optional email field for feedback
Assistant generated file changes: - database.py: Switch to PostgreSQL and add table creation - main.py: Add database initialization --- User prompt: no tables found in replit database. is that normal?
Assistant generated file changes: - main.py: Add immediate logging of search terms --- User prompt: at the point of entering a term, it should be logged in the database. but it does not seem to be the case for the current implementation. why?
Assistant generated file changes: - main.py: Add comments to feedback saving - database.py: Add comments column and parameter, Update save_feedback function --- User prompt: comments not logged in the db. please fix that
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.