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

UpgradeChecker: fix the stable version number #860

Merged
merged 1 commit into from
Jan 28, 2014

Conversation

melissalinkert
Copy link
Member

@ghost
Copy link

ghost commented Jan 21, 2014

This looks fine and good to merge.

However, do we really need to embed a (soon outdated and static) version in the source? Can't we query this externally using the qa server, so the only information we need to hand is our own release version?

@melissalinkert
Copy link
Member Author

The only problem I see with querying the registry is that it would need to return a different version depending upon what the current version is - i.e. the latest version is 4.4.10 if you're running 4.4.9, but it's 5.0.0-rc1 if you're running 5.0.0-beta1.

@joshmoore
Copy link
Member

It's been discussed having both available, but no action has taken place yet (/cc @bpindelski)

@bpindelski
Copy link

@joshmoore The changes in this PR go along the lines with what already is on dev_4_4. Yes, I agree with @rleigh-dundee that ideally the QA server (or any server responding to the upgrade checks) should do some rudimentary logic to detect the major version of the requesting client. At the moment that would necessitate changes in the QA code. Trello ticket?

@melissalinkert
Copy link
Member Author

Knowing that we need to put a pin in registry/QA changes, is there any reason not to merge this before the 5.0.0 code freeze?

@joshmoore
Copy link
Member

Not from me.

melissalinkert added a commit that referenced this pull request Jan 28, 2014
UpgradeChecker: fix the stable version number
@melissalinkert melissalinkert merged commit 1117df6 into ome:dev_5_0 Jan 28, 2014
@melissalinkert
Copy link
Member Author

--rebased-to #872

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants