firebase / firebase/firebase-cpp-sdk

[FR] For macOS, don't unnecessarily stub out iOS+ SDK

オープン
#934 コメント 5 件 リアクション 0 件 担当者 0 名 GitHub で見る
api: core type: feature request
主要言語
C++
スター
326
フォーク
137
平均マージ
3日 9時間
マージ済み PR(30日)
5

説明

Hey wonderful Firebase folks,

The iOS+ SDK these days has [pretty broad support for macOS](https://firebase.google.com/docs/ios/learn-more#firebase_library_support_by_platform). But on quick inspection, it looks like a lot of that functionality, e.g., Analytics, is unnecessarily stubbed out in this C++ SDK.

Could you fix that? It seems like it wouldn't be that hard to change the build to pipe it through, same as tvOS, since the underlying pieces are there.

Thanks for your consideration!
Chris
(ex-Googler)

P.S. If it's already supported--and I've missed it, then please correct me, and my apologies in advance.
P.P.S. #234 is related in that both involve desktop analytics support, but this issue differs in that it should be quite easy to add macOS support more broadly, since it's already supported by an underlying platform SDK.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

No files or tests are named. Start by inspecting the platform conditionals and build configuration for the iOS+ SDK, comparing the macOS path with tvOS; done means macOS builds route supported Firebase functionality through instead of using unnecessary stubs.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp, macos
領域
desktop-dev
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。