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.

Sign up for GitHub

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

Jump to bottom

Feature/python api added two more functions #6292

Merged
merged 2 commits into from
Jun 29, 2017
Merged

Feature/python api added two more functions #6292

merged 2 commits into from
Jun 29, 2017

Conversation

marikaris
Copy link
Contributor

@marikaris marikaris commented Jun 28, 2017 •
edited by bartcharbon
Loading

Added delete_list and upload_zip functionality

Checklist

  • Functionality works & meets specifications
  • Code reviewed
  • [nvt] Code unit/integration/system tested
  • [issue There is no documentation for the python api #6301] User documentation updated
  • [ nvt] (If you have changed REST API interface) view-swagger.ftl updated
  • Test plan template updated
  • Clean commits

@bartcharbon bartcharbon self-assigned this Jun 29, 2017
@bartcharbon bartcharbon self-requested a review June 29, 2017 06:06
@marikaris
Copy link
Contributor Author

Fix #6299

@marikaris
Copy link
Contributor Author

testMolgenisApi.zip
Extract this zip and run to test. It uploads typetest to your localhost:8080 and deletes the first three id's

@marikaris
Copy link
Contributor Author

Fix #6300

@bartcharbon bartcharbon merged commit 42b1c86 into molgenis:master Jun 29, 2017
bartcharbon
bartcharbon reviewed Jun 29, 2017
View reviewed changes
Copy link
Member

@bartcharbon bartcharbon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, works as intended

This was referenced Jun 29, 2017
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bartcharbon bartcharbon bartcharbon left review comments

Assignees

@bartcharbon bartcharbon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marikaris @bartcharbon