nextcloud / socialsharing

🐦 Social sharing via Facebook, Twitter & more
https://github.com/nextcloud-releases/socialsharing
GNU Affero General Public License v3.0
25 stars 15 forks source link

[sharebymail] mailto: link does not contain the email #61

Open rubo77 opened 2 years ago

rubo77 commented 2 years ago

On NC 23 if I share a folder the actual email is missing in the link:

NC_link_missing

tested on Mozilla 94.0 on Ubuntu 21.04

solracsf commented 2 years ago

This seems normal; as you are not entering any email previously, how any email should be filled? 🤔

rubo77 commented 2 years ago

But I entered the email in the field at the top in my screenshot

skjnldsv commented 2 years ago

You changed the label of the share, this is not the same

rubo77 commented 2 years ago

This is not obvious. But ok, Then: if you enter an email as label this email should also be used in the "share to email"-button

skjnldsv commented 2 years ago

No, this is different. Yours is not a mail share, this is a link share. They are not exactly the same.

skjnldsv commented 2 years ago

Here is how you create a mail share image

rubo77 commented 2 years ago

sure, this works the moment, you create the share, but when the share is already created, there is a button in the list, "share to email" which does not contain the email from the share.

Or what is the purpose of this menu-entry?

skjnldsv commented 2 years ago

there is a button in the list, "share to email" which does not contain the email from the share.

This is a different issue then! :) Indeed, if the email exists, the link should most likely contains the email already

Sicoval31 commented 2 years ago

Hi, same problem. After enter a valid email and validate the sharing :

  1. no mail received since the last nextcloud update (no error in logs) (if i use a user's email, the mail is receive, If i use an external mail (yahoo, google, outlook), no mail received)
  2. the "share by mail" not use the mail previously enter ... use no email, the mailto is empty
marcelklehr commented 1 year ago

Creating an email share will cause Nextcloud to send an email on your behalf, AFAIK. This app only adds a link to send a public share link to an email yourself. Thus you have to enter the e-mail address yourself.

rubo77 commented 1 year ago

Why was this closed? The issue is not solved at all. Please read this comment: https://github.com/nextcloud/socialsharing/issues/61#issuecomment-1100960780

skjnldsv commented 1 year ago

@marcelklehr this issue is about using the email from the share in the "send email" link. So that you don't have to type it again in your mail client if it already exists in the nc share :blush: