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

Sendmail #2

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Sendmail #2

wants to merge 4 commits into from

Conversation

F4bsi
Copy link
Owner

@F4bsi F4bsi commented Jun 8, 2019

Hey @andi34
habe die Funktionalität mit dem Mail senden jetzt mal an die neuste Version angepasst und den Code ein bisschen weiter aufgeräumt als in meiner damals genutzten Version.

Problem ist nur ein bisschen, dass das Passwort usw. auch in der Config Datei angegeben wird und wenn man nicht aufpasst auch bei github landen könnte. Vielleicht hast du da ja noch ne bessere Idee?

Auf jeden Fall gerne mal testen :)

Würde, wenn es gut läuft auch mal im Hauptprojekt einen Pull Request erstellen

F4bsi added 4 commits June 8, 2019 23:10
- Added PHPMailer Classes
- Added send per Mail button in index, styles and js
- added sendPic.php file to actually send the image
- Added mail config to config file
- added mail specific translations
@andi34
Copy link

andi34 commented Jun 9, 2019

Super, herzlichen Dank :) ich schaue die Tage rein :)

@andi34
Copy link

andi34 commented Jun 9, 2019

Ist in lib/ einfach eine Kopie von https://github.com/PHPMailer/PHPMailer ?
Wenn ja, könnte man das auch so machen:
.gitignore -> /resources/lib/PHPMailer

und
cd resources/lib
git clone https://github.com/PHPMailer/PHPMailer
cd /var/www/html

So bräuchte man den PHPMailer nicht seperat maintainen.

Edit:
Denke so ist sinnvoll, auch wegen copyright der einzelnen Projekte (MIT und GPL, keine Ahnung wies mit Mischen aussieht)
andi34@6e485b7

@andi34
Copy link

andi34 commented Jun 9, 2019

Bezüglich der config und evtl. Versehentlich öffentlich teilen von Zugangsdaten:
Einfach eine my.config.inc.php laden falls vorhanden und diese über .gitignore ausschließen?

Im README halt den Hinweis rein:
cp config.inc.php my.config.inc.php

Und diese nutzen für die persönlichen Bedürfnisse.

Edit:
andi34@e12ac04

@andi34
Copy link

andi34 commented Jun 9, 2019

Achso, habe die commits in meinem dev2 branch schon Mal gepickt und Formatierung vom Code angepasst (Tab vs Leerschritte). Default einfach auf false schon gesetzt. .gitignore change auch direkt mit in den ersten Commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants