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

Adding feedback option to the tool #61

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

RishikaBhatia17
Copy link
Contributor

The user might face problem while operating the tool or the user might want to notify us by sending suggestions, errors, corrections or updates to us that they want about this tool.
Therefore, a feedback button is added to the app such that the button is visible on every page of the app (since we don't know when the user faces a problem. If he faces a problem in between he can immediately click on feedback button and send notify us with the feedback)
The button is visible on every page of the tool as shown below
image
After we click on the feedback button a pop-up appears as shown below
image
In this pop up there are two fields, one is email field and other is feedback field. In email field user can enter their email id to get response from admin or they can even leave it blank.
In the feedback field the user can give feedback about the tool.
After filling both the fields or only the feedback field the user should click on submit button as shown below.
image
After clicking on the submit button, a popup message appears saying "Thank you for your feedback We value every piece of feedback we receive." as shiown below
image
After clicking on OK button, the user can continue using the tool normally.

PREREQUISITE: before making the change a package has to be downloaded by pasting the following command in terminal: pip install django-tellme

@tonythomas01
Copy link
Owner

tellme looks like a simple first implementation, yes :). The branch needs a rebase, and I would love to take a look!

@RishikaBhatia17
Copy link
Contributor Author

Hi @tonythomas01,
I have updated the branch and resolved the conflicts. I'm working on the custom feedback form and will send a PR soon.
Thanks!

@tonythomas01
Copy link
Owner

I am getting:

    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 956, in _find_and_load_unlocked
ImportError: No module named 'tellme'

on running the patch. You have to add any new requirements to requirements.txt.

@RishikaBhatia17
Copy link
Contributor Author

I have added django-tell package in requirements.txt
sorry for missing it.

@tonythomas01
Copy link
Owner

Okey, I think we have a merge conflict now. Dont know how that came in. Also, I get this on running:

(gdrive-env-3.5.3) ➜  gdrive_to_commons git:(pr/61) ./manage.py migrate 
SystemCheckError: System check identified some issues:

ERRORS:
tellme.Feedback.screenshot: (fields.E210) Cannot use ImageField because Pillow is not installed.
        HINT: Get Pillow at https://pypi.org/project/Pillow/ or run command "pip install Pillow".
(gdrive-env-3.5.3) ➜  gdrive_to_commons git:(pr/61) 

Can you check if we need to add anything more to the requirements.txt ?

@tonythomas01
Copy link
Owner

Thank you. While submitting a feedback, I get this error on the console and I see that the feedback never made it to the database. Can you please check if the reporting actually got logged in the db ?

Screenshot 2020-04-13 at 11 04 45

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.

2 participants