mayukh18 / blindchat Goto Github PK

Description

I was skimming through the code and came across a bug. In the generic template module, there's a GenericTemplate class that contains the add_template function. The function contains a kwarg, buttons, which defaults to an empty list. Using a mutable object as a default argument has unintended consequences and should be avoided.

Recommendation

This is from the official Python docs: Function definitions.
Refer to the definition of the whats_on_the_telly function under this section.

Context

The issue is described in further detail here: Avoid mutable default arguments.

I haven't gone through the entire code base so can't say if this is the only instance or not.

Cheers,
Aditya

Added last_name, first_name, and pic_url variables, in classUser(db.Model), classWaitingListUser, and def add_details(self, name, first_name, last_name,gender,pic_url). This will allow users to enter their first name, last name, and profile picture so they can enter this information so it can be stored in the database and later allow the user to message others.

i've opened it in messanger app
when i clicked on profile ,it show error

I'm new with flask so I don't know if I miss a step following the setup instructions, but everything is working fine except that when I access the url I receive this message instead of the 'connect' page, my chatbot doesn't send anything. 🍶

Not Found
The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.

When using the bot I found the 'Available commands' response was slightly confusing. I believe this could be improved by changing the structure of the returned options, for example:
Currently: 1. quit/exit: quits from the active chat or from the waitlist
Recommended:exit - quits from the active chat or from the waitlist

In my opinion there is no point of the numbering unless this was also accounted when servicing the users input

alias generator is the simplest right now. Needs to be made better.

Problem Description

In app.py, the route /webview/ is configured multiple times, when the code for both views can be essentially put in one. The two views differ in their allowed methods config.

Additional Context

I am working on it.

cc @mayukh18

When I opened it in the messenger app and clicked on the profile it showed me most of the time.

When a new user is encountered ( user doesn't exist in users db ) ask for the age. Simple "What's your age?" Give two options "18 or over" and "under 18". First option will lead directly to the welcome message. The second option will send a message suggesting parental guidance first then go to the welcome message block.

Though the app in under facebook , their policies probably cover this but still it is better to have this check.

A lot of PEP8 errors occur on opening up a Python file in an editor with PEP8 checks enabled.

Fixing these would greatly improve code quality, and help with easy maintenance.

I would like to work on this one, please.

Need to move to a new analytics platform or maybe have a secondary one. Chatbase has good reviews. Opinions required.

I love the project and I will try it out, however, is it fine with you to include contributing file so that I know how I can participate in the application

The link to edit profile (http://embeeblindchat.herokuapp.com/webview/?id=1698188186922882) returns a server error

The profile webview templates/profile.html is too clunky and basic. Needs to be made prettier.

I was trying out the live version of BlindChat and when I chose the option of edit profile and clicked I got Internal Server Error. Maybe the server is too overloaded but I am not sure. Can anyone look at it ? or can we discuss what might be the cause ?

In the app.py file, there is a few lines of code that needed to be taken out. The code that was taken out was:

liked = user.liked

if liked == True:
liked = '1'
else:
liked = '0'