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

Part 1 of many towards chat application #41

Merged
merged 10 commits into from
Dec 22, 2023
Merged

Part 1 of many towards chat application #41

merged 10 commits into from
Dec 22, 2023

Conversation

jeffzheng13
Copy link
Collaborator

@jeffzheng13 jeffzheng13 commented Dec 20, 2023

Quick little add to app logic when user is created that adds an entry to Firestore as well and not just Firebase Auth, also for accounts that already exist but don't have an entry we add one. Entry in database will include user id and user email. Let me know if we want to store first and last name or any other details in these documents @tbeidlershenk.

@jeffzheng13 jeffzheng13 linked an issue Dec 20, 2023 that may be closed by this pull request
@jeffzheng13
Copy link
Collaborator Author

Make sure issue is not closed or just reopen when you merge.

@tbeidlershenk
Copy link
Collaborator

Crash occurred when signing in -
Expected a value of type 'int?', but got one of type 'String'

Relevant stack frame -

Lost-At-Pitt/packages/lost_found_steelhacks/data/item.dart.lib.js 82:254                      fromFirestore
Lost-At-Pitt/packages/lost_found_steelhacks/pages/signup_page.dart.lib.js 3170:31             getItems
Lost-At-Pitt/packages/lost_found_steelhacks/pages/signup_page.dart.lib.js 3148:32             <fn>
Lost-At-Pitt/packages/multiple_stream_builder/src/stream_builder_2.dart.lib.js 206:25         <fn>

Something with the type of the uid?? I'm not sure

Also probably want to also set the first name, last name and default location as part of this (could just default the location to Pitt coordinates for now), could we add that? @jeffzheng13

@jeffzheng13
Copy link
Collaborator Author

Crash occurred when signing in -
Expected a value of type 'int?', but got one of type 'String'

Issue with picture being stored as an int in the database. Changed the default to store as a string which makes more sense. Looking into changing the picture to be stored as a reference but currently it just shows image path stored.

@jeffzheng13 jeffzheng13 self-assigned this Dec 22, 2023
@tbeidlershenk tbeidlershenk merged commit 77da54b into main Dec 22, 2023
1 check passed
jeffzheng13 pushed a commit that referenced this pull request Dec 22, 2023
Part 1 of many towards chat application
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Database improvements
2 participants