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

Browse project #3585

Merged
merged 1 commit into from Mar 9, 2015
Merged

Browse project #3585

merged 1 commit into from Mar 9, 2015

Conversation

jburel
Copy link
Member

@jburel jburel commented Mar 7, 2015

Do not browse project with empty datasets.

see https://trac.openmicroscopy.org/ome/ticket/11952

To test

  • Select a project containing empty datasets.
  • Make sure the browse action is not enabled.

@jburel jburel added the develop label Mar 7, 2015
@pwalczysko
Copy link
Member

Sorry, I am not sure I understand here - does this mean I will not be able to manage my P/Ds in Insight just because the datasets happen to be empty ? How would I delete them then, or how would I fill them in ?

@pwalczysko
Copy link
Member

Aha, maybe it is just the Browse button disabled. I was horrified that I would not be able to expand a Project in the tree if Datasets are empty.

@jburel
Copy link
Member Author

jburel commented Mar 9, 2015

Just the browse button not enabled if empty dataset in project

@mtbc
Copy link
Member

mtbc commented Mar 9, 2015

Works well, good to merge.

joshmoore added a commit that referenced this pull request Mar 9, 2015
@joshmoore joshmoore merged commit 2dc9d1b into ome:develop Mar 9, 2015
@jburel
Copy link
Member Author

jburel commented Mar 9, 2015

--no-rebase

@jburel jburel deleted the browse-action branch March 9, 2015 14:10
@sbesson sbesson added this to the 5.1.0-m5 milestone Mar 19, 2015
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

5 participants