-
-
Notifications
You must be signed in to change notification settings - Fork 95
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
Bug: Fixing the login/ register issue #300
Comments
Congratulations, @ameyakhyati! 🎉 Thank you for creating the issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work! 'We will promptly review your changes and offer feedback. Keep up the excellent work! |
If you have once created your account, a session is created in cookies for the same so you can't re-register unless and until that is cleared. Although it seems like an error but it is the general flow according to Appwrite. If you have a better suggestion on how to streamline this functionality, please propose otherwise I will mark it as close |
The thing is i tried login - in too thinking that my account is created but going to the login section it sent me the message no account found and i need to register |
You can go through the codebase but I don't think there is an issue. If you find any key component, please raise here or in Discord |
Okiee :) |
this issue occurred with me as well. |
Hi @ameyakhyati |
Hello, sorry for not being active , my mid sems are going on , and as for the issue, I am looking into it, and still figuring out how to solve this issue. |
Yeah sure, you can take your time, it's just that to update me on this ticket accordingly |
okie thank you very much :) |
Previously application was supporting old version of appwrite, that's why method written in auth api file was working fine. But someone might have upgraded the appwrite, so package.json contains upgraded version of appwrite. Appwrite have different methods for latest version. It's doesn't support createEmailSession() method, but support createEmailAndPasswordSession() and updateRecovery() have 3 parameters not 4 in upgraded appwrite. After these 2 modification application is working fine for me. |
If I remember correctly, I have removed the dependency updation in package.json so that it will be version locked. Can you confirm whether the current package.json in the master branch has the old version or the new version? |
my appwrite version is 14 and master branch on the repo is 13. But its working in my system. May be if you locked the version and someone made some modification, which is only supported by latest version of the app |
Ohh, thanks for the update. I think we can close this issue then? What's your call? |
If issue is resolved then close it. This issue was not raised by me. |
@Sanchitbajaj02 I sincerely apologize for being inactive and no updating you regarding the issue. |
What happened?
I tried login-in to the website and it gave me errors -
Add screenshots
What browsers are you seeing the problem on?
Brave
Record
The text was updated successfully, but these errors were encountered: