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

7.0 travel purchase alert #21

Closed
wants to merge 26 commits into from

Conversation

joaoalf
Copy link

@joaoalf joaoalf commented Jan 21, 2015

Send alert when a PO linked to the travel is opened

Depends on #20

Joao Alfredo Gama Batista and others added 5 commits January 15, 2015 16:48
* Send an email alert when travel is opened
* Send an email alert when reservation is confirmed
* Add possibility to configure alert recipients
res = {}

for purchase in self.browse(cr, uid, ids, context=context):
if purchase.state == 'approved' and purchase.travel_id is not False:

Choose a reason for hiding this comment

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

This line is too long

@joaoalf
Copy link
Author

joaoalf commented Jan 22, 2015

@bwrsandman the problem is that it depends on #20 and it was not merged yet.

@joaoalf joaoalf force-pushed the 7.0_travel_purchase_alert branch 2 times, most recently from 29b8cac to cf7a7eb Compare January 22, 2015 14:59
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.75%) to 65.17% when pulling cf7a7eb on savoirfairelinux:7.0_travel_purchase_alert into b230577 on OCA:7.0.

@bwrsandman
Copy link

👍

@max3903 max3903 modified the milestone: 7.0 Aug 5, 2015
@ghost ghost force-pushed the 7.0_travel_purchase_alert branch from ce190db to 9221cee Compare August 11, 2015 20:14
dufresnedavid added 2 commits August 12, 2015 10:07
Conflicts:
	travel/i18n/fr.po
	travel/i18n/travel.pot
	travel/travel.py
Joao Alfredo Gama Batista and others added 12 commits August 12, 2015 10:35
If you don't put user_id, this default value will be the id of administrator
and the others users cannot use the filter.
We get an Acl error saying: 'Access denied for ir_filters models'.
Definition of this field:
user_id': fields.many2one('res.users', 'User', ondelete='cascade',
help='The user this filter is private to. When left empty the filter is public'
'and available to all users.)
@ghost ghost force-pushed the 7.0_travel_purchase_alert branch from 9221cee to 8a21867 Compare August 12, 2015 14:42
#, python-format
msgid "Only draft travels can be unlinked"
msgstr "Seuls les déplacements en mode brouillon peuvent être supprimés"

#. module: travel
#: code:addons/travel/travel.py:91
#: help:travel.passenger,message_unread:0 help:travel.travel,message_unread:0
msgid "If checked new messages require your attention."

Choose a reason for hiding this comment

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

correction : "If checked new messages will require your attention"

@pedrobaeza
Copy link
Member

As Travis is red and no fix, I close. Feel free to reopen in case there's interest.

@pedrobaeza pedrobaeza closed this Oct 14, 2016
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.

7 participants