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

[10.0][ADD] event_mail: New module for events mail settings #76

Merged
merged 3 commits into from
May 24, 2017

Conversation

sergio-teruel
Copy link
Contributor

At this time this module only avoid the creation of event.mail records when a event is created.
cc @Tecnativa
cc @chienandalu

@sergio-teruel sergio-teruel force-pushed the 10.0-PR-event_mail branch 2 times, most recently from 08856ca to c355b27 Compare April 19, 2017 23:34
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add "Known issues / Roadmap" section with the pending tasks:

  • Allow to define other mail settings apart from deactivating them.
  • Create event mail templates for having several selectable mail configurations at event level.

@sergio-teruel
Copy link
Contributor Author

@pedrobaeza Please can you add WIP label??

@pedrobaeza
Copy link
Member

Is this already finished?

@sergio-teruel
Copy link
Contributor Author

@pedrobaeza Finished and squas commits. 😄

@pedrobaeza
Copy link
Member

@chienandalu @luismontalba, please review

@rafaelbn rafaelbn added this to the 10.0 milestone Apr 23, 2017
Copy link
Member

@luismontalba luismontalba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionality tested on runbot

@rafaelbn
Copy link
Member

rafaelbn commented May 8, 2017

I cannot test it.. runbot says "502 Bad Gateway nginx/1.6.2"

@sergio-teruel
Copy link
Contributor Author

@rafaelbn Now you cant test it on runbot

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in runbot 👍 . Just to say that you have to activate developer mode (debug) for showing mail schedulers templates

Please @pedrobaeza this is ready..

@pedrobaeza pedrobaeza merged commit f78946c into OCA:10.0 May 24, 2017
@pedrobaeza pedrobaeza deleted the 10.0-PR-event_mail branch May 24, 2017 15:17
ntsirintanis pushed a commit to ntsirintanis/event that referenced this pull request Nov 20, 2018
chienandalu pushed a commit to Tecnativa/event that referenced this pull request May 6, 2019
fuentes73 pushed a commit to Studio73/event that referenced this pull request Oct 14, 2020
ernestotejeda pushed a commit to Tecnativa/event that referenced this pull request Oct 26, 2021
ernestotejeda pushed a commit to Tecnativa/event that referenced this pull request Nov 30, 2021
ernestotejeda pushed a commit to Tecnativa/event that referenced this pull request May 10, 2023
stefan-tecnativa pushed a commit to Tecnativa/event that referenced this pull request Jun 14, 2023
Camille0907 pushed a commit to Camille0907/event that referenced this pull request Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants