• Resolved danielgm

    (@danielgm)


    Right after creating a temp login you show a mailto: link. This one could easily have the actual link put in the body using mailto parameters.

    This seems so handy I would put an additional action icon (e-mail) in the overview that does the same.

Viewing 6 replies - 1 through 6 (of 6 total)
  • @danielgm,

    Thanks for your suggestion. This is in our list and will be incorporated in one of the future release.

    @danielgm

    Just want to update you that this feature is incorporated in Temporary Login Without Password plugin 1.5.7.

    Hope that helps.

    Thread Starter danielgm

    (@danielgm)

    Thanks for the update. I just had a look.

    The way you insert <br> tags in the mail body (using \n in conjunction with PHP’s nl2br) is causing them to be displayed as tags instead of interpreted, at least in Apple Mail.

    I researched it and as per RFC2368, which defines mailto:, it is explicitly stated that the only valid way to generate a line break is with %0D%0A.

    I didn’t want to make that part of the translatable string so I rewrote your code like so:

    $double_line_break = '%0D%0A%0D%0A';	// as per RFC2368
    $mailto_greeting	 = __( 'Hello,', 'temporary-login-without-password' );
    $mailto_instruction  = __( 'Click the following link to log into the system:', 'temporary-login-without-password' );
    $mailto_subject 	 = __( 'Temporary Login Link', 'temporary-login-without-password' );
    $mailto_body    	 = $mailto_greeting . $double_line_break . $mailto_instruction . $double_line_break . $temporary_login_link;

    includes/class-wp-temporary-login-without-password-common.php

    …which works in Apple Mail and also improves on the English language.

    I hope you will use it for the next version.

    • This reply was modified 6 years, 7 months ago by danielgm.
    Thread Starter danielgm

    (@danielgm)

    I guess you could make it even fancier by putting the first name of the recipient, if indicated, between ‘Hello’ and that comma, since it is known to the plugin:)

    @danielgm

    I appreciate your efforts.

    We will test it and incorporate into next release.

    @danielgm

    Just want to update you that we have incorporated a fix suggested by you in Temporary Login Without Password plugin 1.5.8.

    Thank you!

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Paste login link directly into e-mails’ is closed to new replies.