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

Implement Combine support for Firebase #7295

Open
21 tasks
peterfriese opened this issue Jan 13, 2021 · 6 comments
Open
21 tasks

Implement Combine support for Firebase #7295

peterfriese opened this issue Jan 13, 2021 · 6 comments

Comments

@peterfriese
Copy link
Contributor

peterfriese commented Jan 13, 2021

This is a tracking bug for adding Combine support to the Firebase SDKs for Apple's platforms.

If you're interested in contributing code by writing Combine Publishers for one of the SDKs listed below, please leave a comment so we can track interested (and pencil you in on the task list).

Related info

Please check out the following for details about this work stream:

  • DECISIONS.md lists some of our decisions. Not set in stone (and we're always happy to discuss), but serves as a guiding principle
  • DEVELOPING.md contains step-by-step instructions for setting up your development environment for working on Combine Publishers for Firebase
  • README.md contains a description of Combine support for Firebase from a user's perspective. Consider adding a code snippet for the publishers you build to give users an impression of how to use them.
  • CHANGELOG.md is the change log for this module

Roadmap

MVP

  • Implement Combine support for the SDKs marked as P1
  • Submit for API review
  • Address any feedback
  • Merge into the main branch
  • Gather feedback from the wider community

Close the gap

  • Begin work on supporting remaining SDKs
  • API review for this work package
  • Merge into main branch
  • Gather feedback from the wider community

GA

  • Review feedback from the community
  • Address any requests for changes
  • Launch 🚀

SDKs to support

Finished

In Progress

Backlog

Interested in helping out? Leave a comment so we can pencil you in.

  • [P3] Machine Learning
  • [P3] Cloud Messaging
  • [P3] Remote Config
  • [P3] Dynamic Links
@ziadtamim
Copy link
Contributor

@peterfriese I will support Cloud Storage.

@peterfriese
Copy link
Contributor Author

@peterfriese I will support Cloud Storage.

Great - thanks, Ziad. I penciled you in - once you send a draft PR, I will add the PR to our project tracker.

@paulb777
Copy link
Member

paulb777 commented Oct 1, 2021

@peterfriese Should we close and open separate issue for specific items?

@peterfriese
Copy link
Contributor Author

peterfriese commented Oct 1, 2021

Yes, we should do that. I'll spend some time next week to clean up the tracker and set up new issues.

@mortenbekditlevsen
Copy link
Contributor

I still have a PR for initial Combine support in FirebaseDatabase. Let me know if that's of any use:

#7544

@ziadtamim
Copy link
Contributor

There is a PR for Decodable QuerySnapshot publisher #7549

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Combine Support
  
In progress
Development

No branches or pull requests

4 participants