tonythomas01 / gdrive-to-commons

Tool to upload pictures from Google Drive to Wikimedia Commons ⚠️ Moved to Wikimedia Gerrit, so this repo is read only.
https://gdrive-to-commons.toolforge.org/
Apache License 2.0
9 stars 10 forks source link

Adding feedback option to the tool #61

Open RishikaBhatia17 opened 4 years ago

RishikaBhatia17 commented 4 years ago

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 commented 4 years ago

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

RishikaBhatia17 commented 4 years ago

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 commented 4 years ago

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 commented 4 years ago

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

tonythomas01 commented 4 years ago

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 commented 4 years ago

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