Skip to content
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

Closed
2 tasks done
ameyakhyati opened this issue May 12, 2024 · 16 comments
Closed
2 tasks done

Bug: Fixing the login/ register issue #300

ameyakhyati opened this issue May 12, 2024 · 16 comments
Labels
🪲 bug Something isn't working ⭐ enhancement Feature enhancement 🛠 goal: fix The label is related to fixing a bug Invalid This issue or pull request already exists

Comments

@ameyakhyati
Copy link

What happened?

I tried login-in to the website and it gave me errors -

  1. Same name, email ID already exists
  2. Cannot create a session while another session is active but no session being displayed even after refreshing many times

Add screenshots

Palettegram

What browsers are you seeing the problem on?

Brave

Record

  • I agree to follow this project's Code of Conduct
  • I want to work on this issue
@ameyakhyati ameyakhyati added 🛠 goal: fix The label is related to fixing a bug 🪲 bug Something isn't working labels May 12, 2024
Copy link

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!
Kindly remember to check our Contributing Guidelines'

@Sanchitbajaj02
Copy link
Owner

I tried login-in to the website and it gave me errors -

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

@ameyakhyati
Copy link
Author

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

@Sanchitbajaj02
Copy link
Owner

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

@Sanchitbajaj02 Sanchitbajaj02 added 🏆 good first issue Good for newcomers ⭐ enhancement Feature enhancement MEDIUM Difficulty Level: Medium level2 25 points gssoc GSSOC'24 Required Label labels May 12, 2024
@ameyakhyati
Copy link
Author

Okiee :)

@Stiffpixels
Copy link

this issue occurred with me as well.

@yashd-dev yashd-dev mentioned this issue May 16, 2024
2 tasks
@Sanchitbajaj02
Copy link
Owner

Hi @ameyakhyati
What's the status of this issue?

@ameyakhyati
Copy link
Author

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.
If you are okay with me sending you an update with a solution by Sunday night (around 8 ig), then I'd appreciate it very much.

@Sanchitbajaj02
Copy link
Owner

Yeah sure, you can take your time, it's just that to update me on this ticket accordingly

@ameyakhyati
Copy link
Author

okie thank you very much :)

@ArtiGaund
Copy link

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.

@Sanchitbajaj02
Copy link
Owner

Sanchitbajaj02 commented May 22, 2024

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?

@ArtiGaund
Copy link

ArtiGaund commented May 22, 2024

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

@Sanchitbajaj02
Copy link
Owner

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?

@ArtiGaund
Copy link

If issue is resolved then close it. This issue was not raised by me.

@ameyakhyati
Copy link
Author

@Sanchitbajaj02 I sincerely apologize for being inactive and no updating you regarding the issue.

@Sanchitbajaj02 Sanchitbajaj02 added Invalid This issue or pull request already exists and removed MEDIUM Difficulty Level: Medium level2 25 points gssoc GSSOC'24 Required Label labels May 28, 2024
@Sanchitbajaj02 Sanchitbajaj02 closed this as not planned Won't fix, can't repro, duplicate, stale Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 bug Something isn't working ⭐ enhancement Feature enhancement 🛠 goal: fix The label is related to fixing a bug Invalid This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants