anaynayak / buildnotify

A system tray based build status notification app for cctray.xml feeds.

Home Page:https://anaynayak.github.io/buildnotify/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Time Data appears to be off

anaynayak opened this issue · comments

Originally reported by: Anonymous


When using the option to display last build time in the dropdown, the times there are very off.

My CI server is a modified version of cc.rb (though nothing that messes with time, just added some security, and a few css changes).

My server is in UTC, I am in EST.

Changing the drop down in the server configuration dialog does nothing.


Original comment by Robert Cotey (Bitbucket: coteyr, GitHub: coteyr):


Sorry, that was my issue didn't realize I wasn't logged in.

Original comment by Robert Cotey (Bitbucket: coteyr, GitHub: coteyr):


Not used to working on bitbucket or with hg so if I borked something out of alignment, just let me know. But I forked, fixed and submitted a pull.

Original comment by Anay Nayak (Bitbucket: Anay, GitHub: Anay):


Thanks for taking the time to fix this issue. The only question that I had was about the #wrong on https://bitbucket.org/coteyr/buildnotify/changeset/96048722e5324a8e24df46794d9f9cc4f61c80dc#Lbuildnotifylib/projects.pyT121 . I'm guessing you had concerns with the way the url attribute was now being passed ? I'll merge the changes in but am interested in knowing your thoughts.

Original comment by Anay Nayak (Bitbucket: Anay, GitHub: Anay):


Merged pull request #1 based on changed done by Robert. Thanks for the help :)

Original comment by Robert Cotey (Bitbucket: coteyr, GitHub: coteyr):


It was a left over comment, it shouldn't be there. I put www.google.com hard coded as a URL to test that the getter and setter was storing values properly. I stuck a #WRONG there so that I would remember to change it. I changed it, but forgot to move the comment.