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

CODENVY-1009: fix license legality check #1011

Merged
merged 1 commit into from
Oct 24, 2016
Merged

CODENVY-1009: fix license legality check #1011

merged 1 commit into from
Oct 24, 2016

Conversation

ashumilova
Copy link
Contributor

What does this PR do?

Fix license legality check.

What issues does this PR fix or reference?

#1009

Signed-off-by: Ann Shumilova ashumilova@codenvy.com

@ashumilova ashumilova added this to the 5.0.0-M6 milestone Oct 23, 2016
@ashumilova
Copy link
Contributor Author

@riuvshin License legality on nightly5 should be checked, cause now it's false.

@riuvshin
Copy link

@ashumilova nightly5 is not onprem it is saas we can't check this on saas

@ashumilova
Copy link
Contributor Author

Then saas should not have license service deployed, if it's not used, or it should return correct value, like on beta now. Not sure, but the result of this issue #599 may cause it.

Signed-off-by: Ann Shumilova <ashumilova@codenvy.com>
@ashumilova ashumilova merged commit 365663a into master Oct 24, 2016
@ashumilova ashumilova deleted the CODENVY-1009 branch October 24, 2016 16:03
@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Oct 24, 2016

@riuvshin, @ashumilova: https://codenvy.io has Codenvy license and so it returns legalilty=true.
https://nightly5.codenvy-stg.com doesn't has Codenvy license and, as a result it's legalilty=false.
And it doesn't matter are they kind of SaaS or on-prem now.

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