search

anitab-org / mentorship-android

Mentorship System is an application that matches women in tech to mentor each other, on career development, through 1:1 relations during a certain period of time. This is the Android application of this project.
https://github.com/anitab-org/mentorship-android/raw/apk/app-debug.apk
GNU General Public License v3.0
165 stars 379 forks source link

Feat: Allow users to get a copy of their feedback #689

Open Kavya-24 opened 4 years ago

Kavya-24 commented 4 years ago

Is your feature request related to a problem? Please describe. As a user of mentorship-android, I might want to get a copy of the feedback that I provided.

Describe the solution you'd like Extract the text that has been entered as feedback and store it on the database. Use Flask libraries to implement mail features

Additional context May I work on the frontend part of this?

Priyansh-Kedia commented 4 years ago

@Kavya-24 I also thought of this a while back. I think that this would be an issue for the backend and just creating the front end would not suffice as we will not be able to get any information regarding the feedback unless and until the backend does not have the proper data structures to carry it.

What do you think?

Kavya-24 commented 4 years ago

@Priyansh-Kedia What I thought was that if we could store the feedback sent to the organization in some way or the other (a string of text for example) and embed this , we may be able to send the email. I don't know if this is possible as I am not yet familiar with how Flask backend will be working but I think it can be taken into consideration after looking at the following article. https://overiq.com/flask-101/sending-email-in-flask/ @isabelcosta @m-murad @foongminwong Please provide us guidance.

Priyansh-Kedia commented 4 years ago

@Kavya-24 Yes, flask uses SQLAlchemy and SMTP for the delivery of the mails. What I am saying is that I do not think that there is a field in the current database to hold the feedback forms according to the users. Also, the idea of storing messages as a string is good, but how do you think of doing that?

Kavya-24 commented 4 years ago

@Priyansh-Kedia We could create a <String, String> multimap in this, so as to store the email as the key with a particular string associated with it if you are talking about the data structures in particular :)

Priyansh-Kedia commented 4 years ago

@Priyansh-Kedia We could create a <String, String> multimap in this, so as to store the email as the key with a particular string associated with it if you are talking about the data structures in particular :)

That is what I was thinking, you will have to store the key value pair somewhere, and for that you will need API endpoints to coordinate with the backend, and as far as I know, the current backend is not populated with methods for the particular issue. Your issue, nevertheless will be helpful (If it is completed completely).

Please correct me if I am wrong.

somayaj commented 3 years ago

is this available to be worked on? do assign to me if it's available.

kartikeysaran commented 3 years ago

Is this issue open is yes i would like to work on this

epicadk commented 3 years ago

Is this issue open is yes i would like to work on this

I think we would need to implement this in the backend. Not sure why @mayburgos removed the label. @isabelcosta @vj-codes can you confirm?

vj-codes commented 3 years ago

@epicadk yes this is a backend issue , backend issue label was removed because the label was going to be deleted from the list of all labels and as you can see it doesn't exist now. #170 deals for the same on backend.