-
Notifications
You must be signed in to change notification settings - Fork 78
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
fix(Profile flow): Send a contact request (CR) #13598
Conversation
- make `SendContactRequestModal.qml` use the common dialog, use the contact details if we already have it - make some minimal changes to the "Send ID verification" flow since it shares the same dialog - simplify the `CommonContactDialog.qml` footer/buttons handling - adjust the menu item texts - emit toasts when the action is performed - display a tooltip over the compressed elided key Fixes #13518
Jenkins Builds
|
✔️ status-desktop/prs/linux/x86_64/tests-nim/PR-13598#1 🔹 ~5 min 54 sec 🔹 032b6b2 🔹 📦 tests/nim package |
@@ -38,8 +38,8 @@ QtObject { | |||
signal openImagePopup(var image, string url) | |||
signal openProfilePopupRequested(string publicKey, var parentPopup, var cb) | |||
signal openActivityCenterPopupRequested() | |||
signal openSendIDRequestPopup(string publicKey, var cb) | |||
signal openContactRequestPopup(string publicKey, var cb) | |||
signal openSendIDRequestPopup(string publicKey, var contactDetails, var cb) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't want to open a pandora's box, but what if contact details get changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah good point, some of those dialogs already handle that, some don't. My (ultimate) plan is to unify that in the common dialog. Here the reason for passing var contactDetails
thru the signal is simply saving the call when we already have the info
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
What does the PR do
SendContactRequestModal.qml
use the common dialog, use the contact details if we already have itCommonContactDialog.qml
footer/buttons handlingFixes #13518
Affected areas
SendContactRequestModal
Screenshot of functionality (including design for comparison)
Zaznam.obrazovky.z.2024-02-16.13-06-49.webm