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

Order Email: Add ics as attachment and standardize for calendar #7632

Closed
mariobehling opened this issue Jan 19, 2021 · 17 comments · Fixed by #7681
Closed

Order Email: Add ics as attachment and standardize for calendar #7632

mariobehling opened this issue Jan 19, 2021 · 17 comments · Fixed by #7681

Comments

@mariobehling
Copy link
Member

When an email arrives from other event platforms Gmail and other email clients automatically provide an option to add it to the calendar.

  1. Please research and implement the structure of the email and possibly tags, so these emails provide the same functionality.
  2. Add an .ics file as an attachment to order emails for the event as well.

Example how an event shows up on Gmail.

Screenshot from 2021-01-19 10-53-13

@ShauryaAg
Copy link
Contributor

I would like to work on this issue.

@ShauryaAg
Copy link
Contributor

I can't figure out how to set the SMTP_HOST, SMTP_USERNAME etc for development purposes, some help would be appreciated.

@iamareebjamal
Copy link
Member

Set in settings table

@ShauryaAg
Copy link
Contributor

Is there any easy way to do it other than using SQL queries in the Postgres Container?

@iamareebjamal
Copy link
Member

Run frontend and change through it

@ShauryaAg
Copy link
Contributor

I am unable to create a super user for it. I tried running python manage.py initialize_db -c [email protected]:password, but still the user doesn't exist.

Even though the command executes without any exceptions.

image

@iamareebjamal
Copy link
Member

Should have run create_db.py while installing like mentioned in the docs

@ShauryaAg
Copy link
Contributor

ShauryaAg commented Jan 27, 2021

I have installed using docker.

Those docs do not mention to run create_db.py after installation.

@ShauryaAg
Copy link
Contributor

ShauryaAg commented Jan 27, 2021

Even when I run python create_db.py [email protected] password, it doesn't create a new user.

This is the output I get:

/data/app # python create_db.py [email protected] password
/opt/pysetup/.venv/lib/python3.8/site-packages/envparse.py:195: UserWarning: Could not any envfile.
  warnings.warn('Could not any envfile.')
/opt/pysetup/.venv/lib/python3.8/site-packages/envparse.py:195: UserWarning: Could not any envfile.
  warnings.warn('Could not any envfile.')
WARNING:app.instance:Using random secret "45ab1b0c5a8cb8cb2f04a98162b95cbad71d83987dd0b38c25754b3da3dcf95b" for development server. This is NOT recommended. Set proper SECRET_KEY in .env or environment variables
INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
/data/app #

It should've logged "User updated" after creating a new super user.
or at least logged "DB Exception" if it is failing.

@iamareebjamal
Copy link
Member

I don't understand why are you using docker?

@ShauryaAg
Copy link
Contributor

It is easier to setup using docker. Is it not possible to use docker for development purpose?

@iamareebjamal
Copy link
Member

No, it is for deployment only

@ShauryaAg
Copy link
Contributor

Can we create another issue to do necessary changes in order to make docker available for development purposes too?
After solving this issue.

@iamareebjamal
Copy link
Member

What's difficult in local installation?

@ShauryaAg
Copy link
Contributor

There aren't any issues, but docker installation is easier.

@iamareebjamal
Copy link
Member

And development harder

@ShauryaAg
Copy link
Contributor

ShauryaAg commented Jan 28, 2021

Upon some research through here, here, and here. It looks like outlook and some other clients (excluding Gmail) needs Method: Request; in the file headers for it to display the "Add to calendar" and "RSVP" button.

Upon my testing, there is currently no way add that header using Sendgrid or marrow-mail (for SMTP) as of now. However Gmail displays the "Add to calendar" button just by attaching a .ics file.

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

Successfully merging a pull request may close this issue.

3 participants