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

addonbrowser: add LastUpdated window property #6964

Merged
merged 1 commit into from
Apr 18, 2015

Conversation

tamland
Copy link
Member

@tamland tamland commented Apr 16, 2015

Adds a window property to show the last date and time repositories was checked for updates. I'm not sure if 'window properties' is the preferred way to do this. (please comment)

@phil65
Copy link
Contributor

phil65 commented Apr 16, 2015

for me a window property sounds fine.

@ronie
Copy link
Member

ronie commented Apr 16, 2015

perhaps use Updated instead of LastUpdated, for consistency sake.

we use Window(Weather).Property(Updated)

@tamland
Copy link
Member Author

tamland commented Apr 16, 2015

Makes sense:)

@da-anda
Copy link
Member

da-anda commented Apr 17, 2015

should we add the Confluence adjustments along with this PR or do them separately?

@tamland
Copy link
Member Author

tamland commented Apr 17, 2015

If somebody who knows skins can make the adjustments I can cherry pick it, sure

@ronie
Copy link
Member

ronie commented Apr 17, 2015

just merge and i'll handle it afterwards.

@tamland
Copy link
Member Author

tamland commented Apr 18, 2015

jenkins build and merge

@jenkins4kodi jenkins4kodi merged commit d81fd71 into xbmc:master Apr 18, 2015
@hudokkow hudokkow added this to the Isengard 15.0-beta1 milestone Apr 18, 2015
@tamland tamland deleted the last_updated_prop branch August 9, 2015 09:43
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.

None yet

6 participants