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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add useFirebaseFunctions composable #1382

Closed
wants to merge 0 commits into from
Closed

Conversation

luc122c
Copy link
Contributor

@luc122c luc122c commented Jul 7, 2023

Resolves #1327

I've implemented the function you suggested into the source. I've based it on the existing useFirestore composable. Hope this is OK 馃憤

Copy link
Member

posva commented Jul 7, 2023

Thanks! I don't know if it's really useful so I will probably check this later on

@luc122c
Copy link
Contributor Author

luc122c commented Aug 16, 2023

Hmm, I tried to sync my fork with upstream but it seems to have removed the commits.
@posva I'm happy to recover the changes and re-open the PR if you think it will be useful? Otherwise, I'll leave it and #1327 can be closed.

Copy link
Member

posva commented Aug 16, 2023

I still don鈥檛 think this is needed within VueFire. Its better to document it as a cookbook entry

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.

Add useFirebaseFunctions() composable
2 participants