-
Notifications
You must be signed in to change notification settings - Fork 45
Edit Profile and Owner's Info tab fixed #146
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
Conversation
The explore nearby page in itself has some faults for which I'll create other issues after this gets resolved |
Is this error coming after clicking on owner info tab? |
Yes yes @keyurgit45 |
@Mohitmadhav Can you check in the firebase database if the userid field of that book contains the correct userid and is not empty in books collections and the same user is present in the users collection? If present, try adding the same book again |
Yeah, @keyurgit45 there's no field of userid in that book. |
May be these books were added in early stage of the app.. So you can remove these books ig |
I just deleted the whole collection because there were many books like these. |
I can delete the whole user collection also - It already has lot of junk from the past test users |
@keyurgit45 I deleted the user collection too... |
@Mohitmadhav have you deleted accounts from authentication tab on firebase? |
No, just the user collection from Firestore |
Then delete that also otherwise it will create issues |
Yeah I know @keyurgit45 but they have to be deleted one by one so didn't do so for now |
Okay Fine! I am creating new account then Check if (Book info) it is showing on your phone or not . |
I deleted all the accounts from Auth console using website inspect |
@Mohitmadhav I have added books. check if these books are showing on your device correctly. |
Yeah...now the Owner's info is sorted out. |
Description
Summary of the changes and which issue is fixed.
Now Edit profile and owners tab is working. A new profile can be updated. Also in location helper removed the state field instead added street address because we can't get state using geocode package. Existing accounts may not work after this. I tested using a new account, it's working fine.
Fixes #79
Type of change
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
My code follows the code of conduct of this project
My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas