Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 5341722020151296: Issue 2628 - Fix stable version of Yandex.Browser listed on the requirements page (Closed)

Created:
June 3, 2015, 4:24 p.m. by Sebastian Noack
Modified:
June 5, 2015, 12:08 p.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Issue 2628 - Fix stable version of Yandex.Browser listed on the requirements page

Patch Set 1 #

Patch Set 2 : Get both variants and pick newer #

Total comments: 2

Patch Set 3 : Fixed syntax error, removed unneeded list declaration #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -4 lines) Patch
M globals/get_browser_versions.py View 1 2 3 chunks +14 lines, -4 lines 0 comments Download

Messages

Total messages: 6
Sebastian Noack
June 3, 2015, 4:24 p.m. (2015-06-03 16:24:47 UTC) #1
Wladimir Palant
I don't think that the current behavior (stable: manual is newer, beta: manual is older) ...
June 3, 2015, 4:27 p.m. (2015-06-03 16:27:56 UTC) #2
Sebastian Noack
On 2015/06/03 16:27:56, Wladimir Palant wrote: > I don't think that the current behavior (stable: ...
June 3, 2015, 4:37 p.m. (2015-06-03 16:37:45 UTC) #3
Wladimir Palant
http://codereview.adblockplus.org/5341722020151296/diff/5649050225344512/globals/get_browser_versions.py File globals/get_browser_versions.py (right): http://codereview.adblockplus.org/5341722020151296/diff/5649050225344512/globals/get_browser_versions.py#newcode135 globals/get_browser_versions.py:135: response = urllib.urlopen('https://api.browser.yandex.ru/update-info/browser/yandex%s/win-yandex.xml' % (suffix, params)) If I'm not ...
June 3, 2015, 5:11 p.m. (2015-06-03 17:11:35 UTC) #4
Sebastian Noack
http://codereview.adblockplus.org/5341722020151296/diff/5649050225344512/globals/get_browser_versions.py File globals/get_browser_versions.py (right): http://codereview.adblockplus.org/5341722020151296/diff/5649050225344512/globals/get_browser_versions.py#newcode135 globals/get_browser_versions.py:135: response = urllib.urlopen('https://api.browser.yandex.ru/update-info/browser/yandex%s/win-yandex.xml' % (suffix, params)) On 2015/06/03 17:11:36, ...
June 3, 2015, 5:42 p.m. (2015-06-03 17:42:30 UTC) #5
Wladimir Palant
June 3, 2015, 7:25 p.m. (2015-06-03 19:25:14 UTC) #6
LGTM

Powered by Google App Engine
This is Rietveld