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

Confirmation email when adding group members #3

Open
deltafunction opened this issue Jan 26, 2015 · 8 comments
Open

Confirmation email when adding group members #3

deltafunction opened this issue Jan 26, 2015 · 8 comments

Comments

@deltafunction
Copy link
Member

Look for inspiration in user_alias.

@psyllost
Copy link
Contributor

psyllost commented Feb 5, 2015

I used the PHPMailer library, provided by owncloud in 3rd party apps, since we are not able to send mails to dtu accounts, because of a block filter (error: Sender address rejected: Domain not found). I created a gmail account ([email protected]) to send emails via google's smtp. I modified the class.phpmailer.php (more specifically the SmtpSend function) in PHPMailer based on the following link, since there are some changes in the latest version of PHP regarding OpenSSL:
http://www.phpclasses.org/discuss/package/14/thread/76/

Now, mail works.

@DTMA
Copy link

DTMA commented Feb 5, 2015

So people get an e-mail from 'data.deic@gmail'? Couldn't we an address something like '[email protected]:[email protected]'?

Den 05/02/2015 kl. 19.09 skrev psyllost <[email protected]mailto:[email protected]>:

I used the PHPMailer library, provided by owncloud in 3rd party apps, since we are not able to send mails to dtu accounts, because of a block filter (error: Sender address rejected: Domain not found). I created a gmail account ([email protected]:[email protected]) to send emails via google's smtp. I modified the class.phpmailer.php (more specifically the SmtpSend function) in PHPMailer based on the following link, since there are some changes in the latest version of PHP regarding OpenSSL:
http://www.phpclasses.org/discuss/package/14/thread/76/

Now, mail works.


Reply to this email directly or view it on GitHubhttps://github.com//issues/3#issuecomment-73095885.

@psyllost
Copy link
Contributor

psyllost commented Feb 5, 2015

Yes, people can reply to [email protected] and as sender they see "Data DeiC".

@psyllost
Copy link
Contributor

psyllost commented Feb 5, 2015

I can add [email protected] in gmail settings to be the sender but I need the password of this account for that.

@deltafunction
Copy link
Member Author

Please use email-addresses that actually exist...

@psyllost
Copy link
Contributor

psyllost commented Feb 9, 2015

Finally, I got the standard mail php function to work. I added an additional parameter with the senders address. More in documentation:
http://php.net/manual/en/function.mail.php

It seems that php was passing the servers address instead of the email and this address was blocked by dtu, perhaps because it is not a valid email address. I need to make a further research on that.

No need for gmail now. But should I use the "[email protected]" or "[email protected]" as sender?

@psyllost
Copy link
Contributor

At the moment, a group owner can invite another user to the group and a "Pending..." under the name of the invited user appears. The latter receives an email with the text: You have been invited to the group "gid" by "owner". Press the link to accept the invitation.
The link leads to a pop up on MyGroups page, with similar text and the user accepts the invitation and at the same time the "Pending..." disappears from the owner's page.

Things that need to be done:

  • A user should be able to remove himself from a group that has been added. This doesn't work in the production version as well. Seems to be a wrong php call in js, but I will dig into that more in the following days.
  • After I fix that, I will add an option for declining the invitation in the pop up and mail.
  • I will move the pop up in the homepage
  • In the test version hidden_group_owner is not stored in db. Need to find out why
  • Use bootstrap for the popup

Please, let me know if I am forgetting something or additions you prefer.
I will commit to git at a later stage :)

@psyllost
Copy link
Contributor

I haven't put javascript so remember to refresh the page when you test it.

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

3 participants