Merge lp:~mgorven/ibid/timezone-335551 into lp:~ibid-core/ibid/old-trunk-1.6

Proposed by Michael Gorven
Status: Merged
Approved by: Jonathan Hitchcock
Approved revision: not available
Merged at revision: not available
Proposed branch: lp:~mgorven/ibid/timezone-335551
Merge into: lp:~ibid-core/ibid/old-trunk-1.6
Diff against target: 0 lines
To merge this branch: bzr merge lp:~mgorven/ibid/timezone-335551
Reviewer Review Type Date Requested Status
Jonathan Hitchcock Approve
Stefano Rivera Approve
Review via email: mp+16726@code.launchpad.net

This proposal supersedes a proposal from 2010-01-01.

To post a comment you must log in.
Revision history for this message
Stefano Rivera (stefanor) wrote : Posted in a previous version of this proposal

Query: time in gmt
Response: I don't know about the gmt timezone

<place> implies that it'll look up countries, but nothing seems to work:

Response: I don't know about the london timezone
Query: time in united kingdom
Response: *blink*
Query: time in great britain
Response: Sorry...
Query: time in uk
Response: I don't know about the uk timezone
Query: time in gb
Response: I don't know about the gb timezone

review: Needs Fixing
Revision history for this message
Stefano Rivera (stefanor) wrote : Posted in a previous version of this proposal

r'^time\s+in\s+(\S+)$'

It should also match "what is the..."

Revision history for this message
Michael Gorven (mgorven) wrote : Posted in a previous version of this proposal

On Thursday 31 December 2009 19:54:47 Stefano Rivera wrote:
> It should also match "what is the..."
r825

Revision history for this message
Michael Gorven (mgorven) wrote : Posted in a previous version of this proposal

Look what you made me do Stefano!

Revision history for this message
Stefano Rivera (stefanor) wrote : Posted in a previous version of this proposal

any reason why you use different time formats to the rest of the bot?
Look at ibid.utils.format_date

Still some time zone issues:

Query: time in utc
Response: I don't know about the utc timezone
Query: time in UTC
Response: 2010/01/01 12:24:23 SAST is 2010/01/01 10:24:23 UTC
Query: time in uct
Response: I don't know about the uct timezone
Query: time in UCT
Response: 2010/01/01 12:24:48 SAST is 2010/01/01 10:24:48 UCT
Query: time in zulu
Response: I don't know about the zulu timezone
Query: time in Zulu
Response: 2010/01/01 12:24:52 SAST is 2010/01/01 10:24:52 UTC

Otherwise a lot better :)

review: Needs Fixing
Revision history for this message
Michael Gorven (mgorven) wrote : Posted in a previous version of this proposal

Fixed Stefano's issues, and allow source timezones to contain spaces.

Revision history for this message
Stefano Rivera (stefanor) wrote : Posted in a previous version of this proposal

gah, I just tried to invoke it with:

Query: what time is it in paris?
Response: Excuse me?

Revision history for this message
Michael Gorven (mgorven) wrote : Posted in a previous version of this proposal

> gah, I just tried to invoke it with:
>
> Query: what time is it in paris?
> Response: Excuse me?

r831

Revision history for this message
Stefano Rivera (stefanor) wrote : Posted in a previous version of this proposal

I think we're there

review: Needs Resubmitting
Revision history for this message
Stefano Rivera (stefanor) wrote : Posted in a previous version of this proposal

gaah, hwtf did that become resumbit?

review: Approve
Revision history for this message
Michael Gorven (mgorven) wrote :

To make Jonathan and Stefano happy :-P

Revision history for this message
Stefano Rivera (stefanor) :
review: Approve
Revision history for this message
Jonathan Hitchcock (vhata) wrote :

Could we have slightly friendlier responses?

Revision history for this message
Jonathan Hitchcock (vhata) wrote :

Maybe just an "It's" at the front of the time?

review: Approve
lp:~mgorven/ibid/timezone-335551 updated
839. By Michael Gorven

Modify response.

Preview Diff

Empty

Subscribers

People subscribed via source and target branches