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 addition: ability to dump and delete database content #41

Merged
merged 1 commit into from Sep 19, 2016
Merged

Feature addition: ability to dump and delete database content #41

merged 1 commit into from Sep 19, 2016

Conversation

divyenduz
Copy link
Contributor

  • Ability to dump db
    • Used the getSummary interface for taking a dump, we can introduce an import functionality later.
  • Ability to remove db data
    • Made collection.truncate() to return uniformly for mongodb, sqlite collection. We need to make an interface for this should we need to add support for more databases.

Added the following features in this commit
c908d0c 
- Ability to dump db
	- Used the getSummary interface for taking a dump, we can introduce an import functionality later.

- Ability to remove db data
	- Made collection.truncate() to return uniformly for mongodb, sqlite collection. We need to make an interface for this should we need to add support for more databases.
@divyenduz divyenduz mentioned this pull request Jun 21, 2016
Closed
@muatik
Copy link
Owner

muatik commented Jul 6, 2016

@divyenduz sorry for being late. thank you, I will merge this soon.

@divyenduz
Copy link
Contributor Author

No issues, let me know if anything needs to be changed.
On Jul 6, 2016 3:41 PM, "Mustafa Atik" notifications@github.com wrote:

@divyenduz https://github.com/divyenduz sorry for being late. thank
you, I will merge this soon.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#41 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAtj8gMmF-DflHGj2OmhsKhLHw-QndwLks5qS1wIgaJpZM4I6qxg
.

@divyenduz
Copy link
Contributor Author

Hey, long time. Let me know if any thing needs to be implemented from my end on this one. Also, let me know any other issues / features I can take up. 👍

@muatik
Copy link
Owner

muatik commented Aug 1, 2016

I will handle this within two days. for now, nothing is required from your side, thank you again.

@muatik muatik merged commit b1ba787 into muatik:master Sep 19, 2016
@divyenduz
Copy link
Contributor Author

divyenduz commented Sep 28, 2016 •
edited

Resolves issue # #32

@divyenduz divyenduz mentioned this pull request Nov 4, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@divyenduz @muatik