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

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

Merged
merged 1 commit into from Sep 19, 2016

Conversation

@divyenduz
Copy link
Contributor

@divyenduz divyenduz commented Jun 21, 2016

  • 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.
- 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.
@muatik
Copy link
Owner

@muatik muatik commented Jul 6, 2016

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

@divyenduz
Copy link
Contributor Author

@divyenduz divyenduz commented Jul 6, 2016

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

@divyenduz divyenduz commented Aug 1, 2016

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 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 divyenduz commented Sep 28, 2016

Resolves issue # #32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants