-
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(friends): Allow quick add friend from link #906
base: dev
Are you sure you want to change the base?
Conversation
I think this can work to generate link and send to others @Flemmli97 let generatedLink = "";
function generateLink() {
generatedLink = `${window.location.origin}/friends/add/${userId}`;
} |
i mean more like where do we want to provide this functionality? when clicking on copy did button e.g.? |
@Flemmli97 Hey Kevin, I noticed that clicking on "Blocked" button from Friends page redirects to the incoming/outgoing requests instead of Blocked List Grabacion.de.pantalla.2024-12-03.a.la.s.12.08.54.p.m.mov |
@Flemmli97 There is another bug caught by the automation tests - npm run build is failing with the following error:
This is the reason why the tauri build jobs for desktop versions are failing on this branch |
I think would be nice to send in chat for example, the other user just click in add, but would be better, if show pop up, without go out of chats page. |
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.
luis comment needs fixing
i think for chat we already have that if you send a did it will autoformat with button to add (if not we had that in native). so link that redirects you to different page dont think is better than that |
added a dropdown when right clicking copy did in friend page and profile setting page. it will copy a pasteable url |
Download the app installers for this pull request: |
Hello @Flemmli97 is there any update about this one? Its still failing for me. thanks! |
Hey @Flemmli97 so during testing I had an issue So the link will take me to the app and add modal pops up but it will say "unknown user and unknown status message" and will never seem to load the username. But when I click add the request does go through as expected |
yes it just fails to fetch the identity from warp. not too sure why though and i wont pin it on this PR. maybe @dariusc93 can shed some light on it? |
If the user never been discovered before it would attempt discovery upon lookup and would emit an event upon the user being discovered and verified. Not sure if the modal is updating upon the event being emitted either though but it does somewhat make sense on why it goes through since it is happening in the background. If the user been discovered previously by the same account and it is cached but not showing up, i would need more information to see if its a bug in how it is cached. |
it uses a store which updates when identity is fetched from multipass via |
needs shuttle to find users that are not discovered yet (aka warp/wasm update) |
see #893 (review) |
What this PR does 📖
doman/friends/add/<did>
e.g.localhost:5173/friends/add/some_did
.Opening that link brings you to the friends page with a pop up allowing you to send a friend request to that user
Which issue(s) this PR fixes 🔨