Skip to content

Adding deletion and update info #42

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

Closed
wants to merge 1 commit into from
Closed

Adding deletion and update info #42

wants to merge 1 commit into from

Conversation

sadick254
Copy link

The inclusion of how to delete and update are important to help the developer to accomplish CRUD operations.

@chrisvfritz
Copy link

chrisvfritz commented Nov 4, 2016

@sadick254 Deleting and updating are just done the normal Firebase way, rather than anything specific to this library. See the example.

@sadick254
Copy link
Author

I totally agree. Deleting and Updating are done the Firebase way. The main reason is to help someone who is just starting interacting with the js stack (Vuejs,Vuefire & Firebase), to at least get some ideas on how to compose his/her app.

@posva
Copy link
Member

posva commented Nov 4, 2016

@sadick254 Thanks for the initiative. I'd like to update the example to fit coding style and match the example mentioned by @chrisvfritz but I cannot edit the file and add commits to the PR. Did you uncheck the Give access checkbox when creating the PR?

@sadick254
Copy link
Author

@posva my bad i deleted my fork after making the PR i guess thats why you cant make edits.Bad thing i dont know how to reverse that. @chrisvfritz I saw the example but the reason why i didnt want to use that is because there is a small difference between the todo app example and the example already provided on the readme.md.

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

Successfully merging this pull request may close these issues.

3 participants