-
Notifications
You must be signed in to change notification settings - Fork 36
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
Suggest users potential contact sources #3314
base: develop
Are you sure you want to change the base?
Conversation
@osmers Users are expected to click here |
I would not have guessed that :D somehow the sentence leads me to believe that there is an information page (e.g. from the wiki) behind it, that explains, where I can use contacts) - maybe we can rephrase it?
Can you change it to Telefonnummer? :)
So this only shows up with they already have a contact created, that uses the same information, right? Edit: Maybe you are already planning the last point, not sure. Bcs that's what I understood from the description... |
I agree with @osmers. I also only got it because of your explanation. My first suggestion to rephrase it would be simply: |
First of all thank you very much for this PR! It's a cool feature and a cool step forward! I'm truely grateful that you took on such a big task. For now I only tested the usability of the feature and didn't have a look at the code yet. Here are a few things, that I noticed, that didn't make sense to me (yet).
These are my thoughts on this feature for now :) I hope they are helpful in some form or another :) |
Yeah, this is what I meant with my last point as well :) would make a lot of sense to make creating new contacts much easier and faster. |
Done, thank you @JoeyStk |
Applied :) |
10797b0
to
9422c60
Compare
@JoeyStk
|
@JoeyStk
If we ignore edge cases and go without precision, the search can be extended.
Do you mean putting detected phone number and E-mail address into a contact form? |
Ah, thank you for clarifying
I think that the "button"/CTA should be to create a contact from the found information. Therefore I'd rather stick to the icon?
Yeah, right. I also don't have an idea for this 😅
Yes, my idea would be to have button like "Convert to contact" right of the phone number / e-mail address. When I click on the button I get to a contact form and the found information (so for example the found phone number/e-mail address) is already filled out, which makes it easier for me and I don't have to copy/paste anything? I'll try to prepare a commit for this, to make it clear what I mean exactly :)
Yes, that is not what I meant. I think it's a good thing, that the municipalities still have to go through the list of found information, but I would like to make it as easy for them as possible :) |
@MizukiTemma I've now added a commit for the minimal effort "convert to contact" functionality. If you find the time, please let me know what you think about it and feel free to drop it again, if you don't like it :) |
That looks nice 😸 I would name it rather "Create a contact from this information" or something, because the "Convert to contact" sounds as if a contact would be generated automatically and/or the link would be replaced with it. |
e56567f
to
72869da
Compare
Co-authored-by: JoeyStk <[email protected]>
72869da
to
81e9fa3
Compare
@osmers This is the current state. Users find the link "Inhaltsanalyse für bestehende Kontaktdaten" (en: Analysis of content for existing contacts) After clicking it, the suggestion list appears. It looks like this if there is no E-mail address or phone number detected and this, if any suggestion exists. All entries are closed as default. Users can open and see the detail By clicking "Zu Kontakt konvertieren", users will be taken to the contact form and can create a new contact How do you find it? We can also merge this PR (it's anyway available only for us) so Searvice Team can actually test around. note |
@MizukiTemma thank you for all the screenshots and explanation - @JoeyStk took me through the function on his system as well and we tried out a few different things which really helped to get a clearer picture :) |
Finally, here comes the list:
|
We don't know if any of matched contacts is a good replacement of the target E-mail address/phone number. Users maybe want to create a new contact. I'm for leaving "Convert to contact" always.
Ah, I see, the phone number and/or E-mail addresss and/or POI of an embedded contact card are detected as link and listed (both in the contact suggestion and the broken link list). Related to #3280? |
I'm not so sure - I see your point about the system not knowing if the contact is a good match/replacement, but on the other hand, users don't see any progress that way.... |
Better - however I still need to check which contact card belongs to which contact detail - could we maybe style it, so that it works as a table with columns? That would mean that in this example the phone number, which belongs to the second contact card, would show up further down so that it is in line with the beginning of the contact card it belongs to. What do you think? And if there is more than one contact detail that belongs to the card, they should be placed directly below each other (in the same row as the card). If they are found in more than one card, they should also show up more than one, imho. |
Short description
This PR introduces a hint page to help users to create new contacts and replace embedded E-mail addresses and phone numbers with contact cards. This is not an amtomated solution, but I hope it can help and guide users at least. This PR is opened as suggestion of an approach and does not mean to be a finalized something. I'm open to any idea 😸
Proposed changes
*1: not per translation, as the list would be then too long with a lot of duplicates. All E-mail addresses and phone numbers in all languages of the page/event/poi will be shown but without duplicate even if any of them appears multiple times in multiple of translations.
Note: I have decided to show targets (E-mail addresss, phone numbers) per content, not contents per target (where each E-mail address and phone number is used), because 1) users probably need context why and where those contact informations are given while replacing them with contact cards or creating a new contact and 2) the later view is almost the same with broken link list.
Side effects
Those points are still in progress and will be worked on, once this approach receives agreement from the team and we decide for details.
__str__
with link?)Resolved issues
Fixes: #2957
Pull Request Review Guidelines