-
Notifications
You must be signed in to change notification settings - Fork 85
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
SignIn page then contents #1
Comments
Thanks! @sagar-barapatre for opening this issue, I found it quite interesting, but don't you think that this extra step of signing in every time, would only increase the complexity for the student, who just wants to search for the teacher? |
I think that the whole data regarding a particular teacher would be leaked including phone number. So in that case a random user can see all the teachers and get the phone numbers directly displayed on the website and can contact that particular teacher, then what's the use of this website, I think that should be done so as to provide proper services to person who really needs it. |
@sagar-barapatre would you like to work on this? |
Yeaaaah ! definitely |
Another approach could be to show contact details only after signin and basic details directly, without signin. WDYT? |
Yeah that can also be done! |
@sagar should this be assigned to someone else? |
@sagar-barapatre please assign this issue to me under gssoc 21 |
@Namanl2001 I would like to work on this issue. I think first google signin and then contents would work. |
@Namanl2001 We can show other information of teacher (except address and contact details) if the user is not signed in. And if the user is signed in we show him/her the other details too. |
May be we can first authenticate the user by either signup/ signin then and then only we'll show the contents of our website.
Here we are asking to sign in through google and the anonymous user can see the complete website without sign-in .
So what I think we can do is in the homepage we will directly prompt the user to first signup/signin, then and then only we'll authenticate them to see the contents of the website.
Let me know if you find this interesting.
The text was updated successfully, but these errors were encountered: