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

A few metadata-mgmt app fixes #80

Merged
merged 8 commits into from
Apr 28, 2015
Merged

Conversation

paterczm
Copy link
Contributor

Addresses 3 issues:

Note that version sorting is done alphabetically, i.e. 0.10 will come before 0.2. It's not perfect, but hopefully sufficient.

There are no tests. See #79.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 22.22% when pulling fad6aa6 on paterczm:todos into 74ae58c on lightblue-platform:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 22.22% when pulling fad6aa6 on paterczm:todos into 74ae58c on lightblue-platform:master.

@paterczm paterczm changed the title Todos A few metadata-mgmt app fixes Mar 31, 2015
@dcrissman
Copy link
Member

Is there any way I could test these changes before approving?

@dcrissman
Copy link
Member

@paterczm you will need to merge in changes from master

Marek Paterczyk added 2 commits April 27, 2015 14:51
@coveralls
Copy link

Coverage Status

Coverage remained the same at 22.22% when pulling aee291c on paterczm:todos into 04f9e56 on lightblue-platform:master.

@paterczm
Copy link
Contributor Author

Openjdk7 build got stalled twice in a row. But openjdk8 passed - good enough?

@dcrissman
Copy link
Member

@paterczm looks like this PR needs a master merge.

Conflicts:
	common/src/main/webapp/js/lightblue-apps.js
@coveralls
Copy link

Coverage Status

Coverage remained the same at 17.39% when pulling 71814fa on paterczm:todos into 95fd53a on lightblue-platform:master.

dcrissman added a commit that referenced this pull request Apr 28, 2015
A few metadata-mgmt app fixes
@dcrissman dcrissman merged commit 8974e96 into lightblue-platform:master Apr 28, 2015
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

3 participants