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

Show filters only for logged in users. #1437

Merged
merged 2 commits into from Oct 4, 2016

Conversation

jkmarx
Copy link
Member

@jkmarx jkmarx commented Sep 29, 2016

Resolves #1316

@jkmarx jkmarx added this to the v1.5.0 milestone Sep 29, 2016
@codecov-io
Copy link

codecov-io commented Sep 29, 2016

Current coverage is 33.95% (diff: 100%)

Merging #1437 into develop will not change coverage

@@            develop      #1437   diff @@
==========================================
  Files           328        328          
  Lines         22755      22755          
  Methods           0          0          
  Messages          0          0          
  Branches       1218       1218          
==========================================
  Hits           7727       7727          
  Misses        15028      15028          
  Partials          0          0          

Powered by Codecov. Last update cd66d00...2dcc813

@scottx611x scottx611x merged commit ddd46c4 into develop Oct 4, 2016
@scottx611x scottx611x deleted the jkmarx/launchpad-hide-filter branch October 4, 2016 14:29
jkmarx pushed a commit that referenced this pull request Oct 4, 2016
* Fix #1312

* Enable share button only for data set owners. (#1434)

* Jkmarx/sharing modal enhancements (#1436)

* Enable share button only for data set owners.

* Add helpful text of sharing modal.

* Adjust text.

* Group API returns group uuid.

* Add link to groups in sharing modal.

* Add group link to ui.

* Add border.

* Show filters only for logged in users. (#1437)

* Show filters only for logged in users.

* Fix spacing.

* Mccalluc/remove sphinx (#1438)

* Clear out docs directory

* Remove sphinx dependencies
Nothing else seems to have come in with these commits

* Jkmarx/collaboration add member enhancements (#1440)

* Add form validation.

* Reformat modals to make consistent.

* Fix styling.

* Reformat code.

* Add module unit test.

* Add close x to each modal.

* Add alerts modal.

* Add comments.

* Email invite unit test.

* Add alert messages to add-group.

* Add enter capability for enter.

* Fix styling.

* Rename method for clarity.

* Add unit test.

* Shorten time.

* Add close to edit group.

* Adjust layout and css.

* Adjust css on group member edit.

* Jkmarx/data import slug validation (#1441)

* Remove duplicat file.

* Fix div error.

* Add error message for invalid slug.

* Round percentages. (#1442)

* Hackdna/workflow import (#1443)

* Refactor get_workflows()

* Refactor import_workflow() and check_workflow_inputs()

* Code and string formatting

* Fix a typo

* Avoid relying on description of step inputs in workflow JSON for workflow import and execution

* Remove unused get_step_inputs() and get_input_steps()

* Improve code style and simplify logic

* Remove unused function get_workflow_inputs()

* Remove unnecessary conditional and variable

* Remove unnecessary variable workflow_relationships, reformat code

* Raise an exception instead of returning None

* Improve code style

* Remove unused code

* Avoid redundant UUID generation

* Fix log message handling

* PEP8 formatting

* Remove unused code

* PEP8 fixes

* Jkmarx/dataset2 new layout (#1413)

* Remove tab buttons.

* Remove other orange button references.

* Refactor, remove style.

* Remove style tag.

* Remove unused states.

* Remove hanging div.

* Add conditional for dataset 2.

* Move analysis tab to it's own partial.

* Move details tab to own partial.

* Move files to own partial.

* Tab views.

* Fix unit test.

* Move display into own component.

* Set default to files.

* Add unit test.

* Remove unused code.

* Render pivotMatrixView call from within directive.

* Unwrap provenance-view-tab watcher.

* Add watchers to directive.

* Remove console.log.

* Correct description.

* Update label.

* New launch analysis component.

* Add disable button conditionals.

* Add unit test.

* Add save directive.

* Move save and reset buttons.

* Add unit tests.

* Move selected nodes text.

* Move file display and config to new layout.

* Add view and remove partial.

* Add comments.

* Adjust popover location.

* Move save funtion to node_group partial.

* Rename reset to better reflect purpose.

* Rename partial.

* Move analyses tab into own feature.

* Column adjustments.

* Parent controller.

* Move file to view folder.

* Keep only table view.

* Fix typo in address.

* Change buttons size.

* Fix hidden popover.

* Add css styling.

* Fix padding.

* Fix padding

* Fix unit test.

* Fix reset button.

* Remove console.

* Fix reset selection count.

* Fix scoping issue.

* Remove console.log

* Fix run analysis ui.

* Update color scheme.

* Jkmarx/dataset2 igv (#1433)

* Remove tab buttons.

* Remove other orange button references.

* Refactor, remove style.

* Remove style tag.

* Remove unused states.

* Remove hanging div.

* Add conditional for dataset 2.

* Move analysis tab to it's own partial.

* Move details tab to own partial.

* Move files to own partial.

* Tab views.

* Fix unit test.

* Move display into own component.

* Set default to files.

* Add unit test.

* Remove unused code.

* Render pivotMatrixView call from within directive.

* Unwrap provenance-view-tab watcher.

* Add watchers to directive.

* Remove console.log.

* Correct description.

* Update label.

* New launch analysis component.

* Add disable button conditionals.

* Add unit test.

* Add save directive.

* Move save and reset buttons.

* Add unit tests.

* Move selected nodes text.

* Move file display and config to new layout.

* Add view and remove partial.

* Add comments.

* Adjust popover location.

* Move save funtion to node_group partial.

* Rename reset to better reflect purpose.

* Rename partial.

* Move analyses tab into own feature.

* Column adjustments.

* Parent controller.

* Move file to view folder.

* Keep only table view.

* Fix typo in address.

* Change buttons size.

* Fix hidden popover.

* Add css styling.

* Fix padding.

* Fix padding

* Fix unit test.

* Fix reset button.

* Remove console.

* Fix reset selection count.

* Fix scoping issue.

* Remove console.log

* Fix run analysis ui.

* Update color scheme.

* Add IGV directory.

* Add directive.

* Include partial in tab.

* Add css.

* Disable if no nodes are selected.

* Add error handling and code to accomodate data set2.

* Fix unit test and adjust css.

* Add modal for launching igv

* Add modal script.

* Add visualization directory.

* Add module.

* Update partials.

* Remove unused code.

* Add script to partial and launch modal when igv selected.

* Remove inline script.

* Restructure modals to launch when visualization is selected.

* Remove unused service.

* Remove ununeccessary ctrl.

* Clean up responsibilities.

* Add conditions to launching igv.

* Add factory.

* Add comments.

* Controller unit test.

* Add visualization ctrl test.

* Add directive unit tests.

* Add factory unit test.

* Comments.

* Fix filter bug.

* Fix description.

* Remove duplicate id.

* Add method comment.

* Fix unit test.

* Remove unused dependency.

* Fix logger error message.

* Clafity variable name.

* Add log error message.

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

Successfully merging this pull request may close these issues.

None yet

3 participants