-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(fdc): Initial Release of Data Connect #13313
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* First pass at dataconnect * Got mutations to work * Finished WIP * Used generated SDK * Removed unrelated chagnes * Cleanup * Updated deps * Addressed comments * Removed prints * Removed unnecessary protos * Removed user class * Removed unnecessary class * Removed unnecessary deps * Added back config files * Fixed formatting * added index.html * Added license files * Fixed formatting * Fixed linting issues * Fixed formatting * Added protobuf as dependency * Added license * Ignored example dir for secret checking * Added changelog and readme * Updated package version * Implemented app check * Removed key * Revert "Removed key" This reverts commit 6a643f1. * Removed key again * Added error catching implementation (#13163) * feat(dataconnect): Added error handling (#13175) * Added missing file * Removed unnecessary changes * Miscellaneous cleanup * Removed macros experiment * Removed .then statements * Removed debug prints * Addressed first pass of comments * Added movie_insert gql file * Added comments * Renamed package * Another package rename * Removed common package * Added missing files * Updated readme * Revert "Removed common package" This reverts commit 422bd80. * Various improvements * Made changes per api proposal * Fixed initialization issues with generated SDK * Made changes according to api council review * Addressed Comments. Removed references to personal firebase projects Moved mapped host logic to new firebase_common package * Added missing license header to files * /s/query/operation * API Council Changes * Included list movies changes * API council updates * Addressed comments * Removed print * Added license header * Removed files that should be gitignored * Removed requirement on firebase_options * Added missing line * Removed windows and linux folders, updated changelog and gitignore * Updated version of firebase_data_connect
* First pass at dataconnect * Got mutations to work * Finished WIP * Used generated SDK * Removed unrelated chagnes * Cleanup * Updated deps * Addressed comments * Removed prints * Removed unnecessary protos * Removed user class * Removed unnecessary class * Removed unnecessary deps * Added back config files * Fixed formatting * added index.html * Added license files * Fixed formatting * Fixed linting issues * Fixed formatting * Added protobuf as dependency * Added license * Ignored example dir for secret checking * Added changelog and readme * Updated package version * Implemented app check * Removed key * Revert "Removed key" This reverts commit 6a643f1. * Removed key again * Added error catching implementation (#13163) * feat(dataconnect): Added error handling (#13175) * Added missing file * Removed unnecessary changes * Miscellaneous cleanup * Removed macros experiment * Removed .then statements * Removed debug prints * Addressed first pass of comments * Added movie_insert gql file * Added comments * Renamed package * Another package rename * Removed common package * Added missing files * Updated readme * Revert "Removed common package" This reverts commit 422bd80. * Various improvements * Made changes per api proposal * Fixed initialization issues with generated SDK * Made changes according to api council review * Addressed Comments. Removed references to personal firebase projects Moved mapped host logic to new firebase_common package * Added missing license header to files * /s/query/operation * API Council Changes * Included list movies changes * API council updates * Addressed comments * Removed print * Added license header * Removed files that should be gitignored * Removed requirement on firebase_options * Added missing line * Removed windows and linux folders, updated changelog and gitignore * Updated version of firebase_data_connect * test(fdc): add e2e tests to the FDC package * add e2e tests to CI * test generated content * update * test * add postgres * more generation tests * update podfile * fix analyze * regenerate with latest version * add more queries * add more queries? * removing not generated commands * more unit tests * timestamp * licence * analyze * add dev dep * update generated * ignore generated for licence * add .firebaserc for dataconnect setup * fix ci * fix setup * test * more testing * improve * unset PGSERVICE * mote testing * okay? * maybe this? * update pubignore * update * ?? * hop * more * hophophop * more * web * tests * test * merge clean * regenerate * fix * fix * more tests * remove GoogleInfo Plist * fix options * test * fix emulator * clear text * wut? * fix generated code * public level * fix * fix generated * test * maybe * alors peut etre * tests done * update caching * e2e tests * tests done * fix * fix concurrency * fixes * fix * again * tests update * test --------- Co-authored-by: Maneesh Tewani <mtewani@google.com> Co-authored-by: Maneesh Tewani <maneesht@users.noreply.github.com>
Lyokone
changed the title
feat(fdc): Initial Release of Data Connect (#13063)
feat(fdc): Initial Release of Data Connect
Sep 12, 2024
russellwheatley
requested review from
Salakar and
russellwheatley
as code owners
September 17, 2024 08:55
SelaseKay
approved these changes
Sep 17, 2024
russellwheatley
approved these changes
Sep 17, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR aims at adding Firebase Data Connect.
Checklist before merging: