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

[Story] Search a user to start a chat #1055

Closed
9 tasks done
Tracked by #988
alfogrillo opened this issue Feb 24, 2023 · 5 comments
Closed
9 tasks done
Tracked by #988

[Story] Search a user to start a chat #1055

alfogrillo opened this issue Feb 24, 2023 · 5 comments

Comments

@alfogrillo
Copy link

alfogrillo commented Feb 24, 2023

As a user I want to find other people to chat with even If I don't have their matrix id

Scope

Engineering

Preview Give feedback
  1. 4 of 4
    A-DMs AirFocus - EX Feature - Story App: ElementX Android App: ElementX iOS T-Enhancement T-X-Platform Task Team: Element X Feature X-Needs-Rust
  2. 2 of 2
    A-Search App: ElementX Android App: ElementX iOS T-X-Platform Task Team: Element X Feature
  3. 4 of 4
    A-Search App: ElementX Android App: ElementX iOS T-Enhancement T-X-Platform Task Team: Element X Feature

Out of scope

  • Nothing at the moment

Sign-off

Android

Preview Give feedback

iOS

Preview Give feedback
@kittykat
Copy link
Contributor

kittykat commented Apr 4, 2023

Must be tested with test accounts for now

@kittykat kittykat added the X-Needs-Signoff Stories and Epics which are ready for review by product, design and QA label May 26, 2023
@ghost
Copy link

ghost commented May 31, 2023

On iOS this seems mostly okay. I have filed one issue (#1712) but I'm not sure it directly relates to this user story. I don't think that issue should block QA sign off for public testing on iOS.

However Kat's comment last month about only using test accounts should block. In my opinion a potential for data loss is not acceptable, even for public testers who accept a certain amount of risk.

I still need to test this on Android.

@Johennes
Copy link
Contributor

Johennes commented Jun 1, 2023

However #1055 (comment) month about only using test accounts should block. In my opinion a potential for data loss is not acceptable, even for public testers who accept a certain amount of risk.

I believe this should be obsolete by now. It only affected Android.

@Johennes Johennes closed this as completed Jun 8, 2023
@Johennes
Copy link
Contributor

Johennes commented Jun 8, 2023

Closing this since we've reviewed it in the weekly on Tuesday and shipped it in the iOS TestFlight today. Any further issues can be tracked under the epic which I'm keeping open until QA has processed the whole scope.

@ghost
Copy link

ghost commented Jul 4, 2023

Retested this on Element X iOS 1.1.8 (270) and Element X Android 0.1.0-nightly (July 1, 2023). It seems to be working in line with the acceptance criteria. Removing this from my backlog, but please tag me if you need something tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants