-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
idea: Allow invoice sending #454
Comments
Hello, I'd like to tackle this issue, is it still available? |
Yeah sure thing @Domin1kG. Feel free to to checkout my responses to some other recent issues to get a few more details of what I'd expected when adding a feature. Thanks! |
Great, I'll check them and keep you informed on my progress and possible solutions, before I make any changes to the code. |
Hello, I also wanted to make sure I understood the problem correctly, so I need to make it possible to send the invoice via email as a PDF attachment? Sorry if I misunderstood something, I'd love to know your opinion, so we are on the same page. I also had an error thrown with the AWS_ENABLE variable not being present while starting up the server, so I couldn't try the email function in the GUI, I'm not sure if I need to setup an access key in helpers.py file, or if something else is messing things up. If perhaps I missed something, or some important files related to this issue, or have any suggestions of what I should look into, I'd be more than happy to know. |
Hey @Domin1kG, Thanks for taking an interest in this issue, much appreciated! Just a quick question, do you have discord? If not that's no problem, i'll send a message below with answers to the questions :) |
Hello, yes I do, I am the one who joined about an hour ago on your server, so feel free to DM me and thanks for the quick reply :) |
We currently have two "email" features:
The new feature for this issue was to merge this emailing with Invoices. Basically so we can email OUR users and THEIR users when actions are done, e.g. an invoice was created, an invoice was marked as over due.
We currently don't do any PDF generation on the backend as the implementation proved to be difficult. Me and domejko spent quite a few hours attempting but ended up dropping it and let browsers handle it, so no we cannot attach them.
Not 100% sure what you mean with this, but yeah this feature should be a modal that pops up giving the user the chance to input a subject, email body, TO address, CC and BCC. Some limits should be made, but also be configurable as some enterprise may need different limits.
Of course no problem, thank you for asking!
Yeah, the AWS setup is a MESS right now. As it's currently really just made for me as i'm running it in production but some of it isn't open source. |
Great, thank you very much, I'll think through how could this be implemented and let you know beforehand. I'll also try to work out the AWS, so I can see the and current layout for emails and think of how it can be altered to suit the needs for this issue. If I run into a problem I can't deal with, can I contact you through your discord account, so I don't make this issue too long? |
Of course no problem. Also for emails, you don't actually need to have them setup. If there is no email setup in the environment variables then it just prints them to the console. So if you just turn off all AWS flags/variables then you should be good to go. |
How big is this feature?
Medium
Describe the problem
Describe a possible solution
No response
The text was updated successfully, but these errors were encountered: