OpenERP Community Backports (Addons)

Merge lp://staging/~andrei-levin/ocb-addons/6.1 into lp://staging/ocb-addons/6.1

  1. 6.1
  2. Merge into 6.1
Proposed by Andrei Levin
Status: Needs review
Proposed branch: lp://staging/~andrei-levin/ocb-addons/6.1
Merge into: lp://staging/ocb-addons/6.1
Diff against target: 67 lines (+21/-4)
2 files modified
base_calendar/base_calendar.py (+20/-3)
base_calendar/wizard/base_calendar_invite_attendee.py (+1/-1)
To merge this branch: bzr merge lp://staging/~andrei-levin/ocb-addons/6.1
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza code review Approve
Review via email: mp+222862@code.staging.launchpad.net

Description of the change

Timestamps in body part of the meeting invitation email displays time in UTC and not in the user's timezone.

Steps to reproduce:
1) Set the timezone in user's preference,
2) Install CRM module and create a meeting and run `Invite People` wizard with `Send mail?` option marked.

You wont have any timezone information in invitation email contents.

This fix will consider timezone in User Preferences and will show timestamps with timezone information in email body.

The FIX is created by Ravi Gohil

To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Hi, Andrei,

Thanks for bringing this module to 6.1.

Some little things to fix:

- Don't change module version. Although it could be convenient to reflect on module version the changes, neither OpenERP nor OCA do it usually. One of the reasons for OCA to not do that is that we are going to desynchronize versions numbers between official and OCB modules.
- Don't change .bzrignore.
- If original author id Ravi Gohil, you can credited him when you make the commit, adding argument --author and his e-mail, that you can get it querying with 'bzr log <ravi_branch> -r -1'.

Regards.

review: Needs Fixing (code review)
Reply
lp://staging/~andrei-levin/ocb-addons/6.1 updated
6844. By Ravi Gohil (OpenERP) (rgo-openerp)

[FIX] base_calendar: Timestamps in body part of the meeting invitation email shows time in UTC. After application of this fix, timestamps will look like '2012-11-07 11:30:00 PM IST', which considers timezone from user preferences : (Maintenance Case : 581633)

Revision history for this message
Andrei Levin (andrei-levin) wrote :

Hi Pedro,

I hope it's OK now. I uncommitted and recommitted the code. If it
continue this way I'll become Bzr guru :-)

Regards

Andrei

2014-06-16 8:24 GMT+02:00 Pedro Manuel Baeza <email address hidden>:
> Review: Needs Fixing code review
>
> Hi, Andrei,
>
> Thanks for bringing this module to 6.1.
>
> Some little things to fix:
>
> - Don't change module version. Although it could be convenient to reflect on module version the changes, neither OpenERP nor OCA do it usually. One of the reasons for OCA to not do that is that we are going to desynchronize versions numbers between official and OCB modules.
> - Don't change .bzrignore.
> - If original author id Ravi Gohil, you can credited him when you make the commit, adding argument --author and his e-mail, that you can get it querying with 'bzr log <ravi_branch> -r -1'.
>
> Regards.
> --
> https://code.launchpad.net/~andrei-levin/ocb-addons/6.1/+merge/222862
> You are the owner of lp:~andrei-levin/ocb-addons/6.1.

--
Didotech Srl

Via T.Aspetti, 248
35133 Padova (PD)

Tel 049 8592286
Cell.: 347-2426694
www.didotech.com
www.simplerp.it

Reply
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Thank you very much for the changes. I have searched on another branches and his e-mail is: <email address hidden>, but we can put it on merge time, so don't worry more about it.

Regards.

review: Approve (code review)
Reply

Unmerged revisions

6844. By Ravi Gohil (OpenERP) (rgo-openerp)

[FIX] base_calendar: Timestamps in body part of the meeting invitation email shows time in UTC. After application of this fix, timestamps will look like '2012-11-07 11:30:00 PM IST', which considers timezone from user preferences : (Maintenance Case : 581633)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches