-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat(chat): impl file sharing chat #704
Conversation
Download the app installers for this pull request: |
@Flemmli97 Im getting some weird behavior when trying to test this. Sometimes the attachment will send through but most times it will not. Ive tried this on Safari/Chrome/Brave Ive done all new accounts, Ive also tried refreshing the accounts if need be When the attachment doesnt send it just sends a blank bubble in the chat and this error appears in console
Screen.Recording.2024-10-15.at.12.44.41.PM.mov |
@Flemmli97 Retested with Sara and we both had a lot of issues When you try to send a file it doesnt seem to be sending correctly. User will get a notification like they got a new message but when they go to chat no new message appears only the banner that says "new message" After the sender sends a file it kinda breaks the entire app. User cannot use the nav bar to navigate anymore, if they refresh chat either endlessly loads or just loads a black screen. It was throwing these errors in logs for both Sara and I Screen.Recording.2024-10-17.at.12.08.29.PM.mov |
@Flemmli97 Automation test caught the following bug:
|
fixed. was a bug from merging |
What this PR does 📖
Which issue(s) this PR fixes 🔨