Skip to content
This repository has been archived by the owner on Aug 27, 2022. It is now read-only.

[BUG]: When sending E-Mail and "Send me a link to all pictures in the next few days" is enabled, no E-Mail is sent #461

Closed
talentlos opened this issue Jul 6, 2022 · 3 comments · Fixed by #462
Labels
bug Something isn't working

Comments

@talentlos
Copy link

Describe the bug

When using the E-Mail feature with option "Send me a link to all pictures in the next few days" is enabled, no E-Mail will be sent, unless the checkbox is unchecked. Mail-Adresses are stored correct.

Steps to reproduce the behavior

Use E-Mail-Feature in Gallery

Expected behavior

When Checkbox "Send me a link to all pictures in the next few days" is enabled and Button "Send Mail" is pressed, mail-adress should be stored in file and E-Mail should be sent.

Photobooth Version

latest dev

Browser and browser version

all browsers

Operating System

Raspberry Pi OS with desktop

Did it work before?

Yes

Latest working version

No response

Additional context

No response

@talentlos talentlos added the bug Something isn't working label Jul 6, 2022
@andi34
Copy link
Owner

andi34 commented Jul 6, 2022

Hey and thanks for the feedback. The button has the wrong text, it would need to be called "add address to database" or similar. It's not meant to send an email and should only be used if the single picture can't be send directly. I am going to adjust the button asap.

@andi34
Copy link
Owner

andi34 commented Jul 6, 2022

Description of the option from the manual at http://localhost/manual (also tooltip on hover):

If enabled, entered email adresses get stored inside a database instead sending choosed picture directly. For example, this can be used to send a mail with all pictures or a link to an online gallery after the event.

@talentlos
Copy link
Author

talentlos commented Jul 8, 2022 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants