-
Notifications
You must be signed in to change notification settings - Fork 409
feat(dc): Add executeQuery and executeMutation APIs to Data Connect #2979
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
Open
stephenarosaj
wants to merge
39
commits into
master
Choose a base branch
from
rosa/impersonate
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,996
−345
Open
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
524d7d8
add in changes from stephenarosaj/fdc-impersonate
stephenarosaj bc92c5e
finish adding in changes from stephenarosaj/fdc-impersonate
stephenarosaj d43588e
update Google Inc. to Google LLC, run npm install; npm run build
stephenarosaj dcd493f
run npm apidocs
stephenarosaj 23fe1f4
remove public execute apis
stephenarosaj ae8096a
convert executeOperation api to OperationRef(...).execute() api
stephenarosaj 52a18d7
remove internal client from operation refs
stephenarosaj 3cb6645
cleanup javadocs to address workflow failures
stephenarosaj bad9808
npm run apidocs
stephenarosaj 6bdef60
spread GraphqlOptions arguments in OperationRefs and executeOperation…
stephenarosaj 798c2dc
convert unit tests to use spread args
stephenarosaj 85a6b4b
convert integration tests to use spread args
stephenarosaj 5f34343
add executeQuery test cases which do not provide impersonation option…
stephenarosaj 088d882
add executeMutation test cases which do not provide impersonation opt…
stephenarosaj 216b3ac
run npm apidocs
stephenarosaj fb5a3de
address try/catch comment
stephenarosaj fd4ffb5
address await and reject grouping comment
stephenarosaj b118905
address getUrl comments
stephenarosaj cf72f38
address insecureReason comment
stephenarosaj 43f223d
convert autopush resources to prod
stephenarosaj 2c642fb
add RefOptions, [Operation,Query,Mutation]Ref, [Operation,Query,Mutat…
stephenarosaj 12f1258
revert OperationRef.execute() API to executeOperation API
stephenarosaj 60649e4
revert OperationRef.execute() API to executeOperation API
stephenarosaj 918c4a0
revert tests to use DataConnect.executeOperation() API instead of Ope…
stephenarosaj 4dad488
revert package version
stephenarosaj 786c75f
merge master into rosa/impersonate
stephenarosaj 89e691a
update executeOperation API to return executeOperationResponse
stephenarosaj 5721657
update comments
stephenarosaj fd61f1a
add invalidateAdminArgs to handle variadic JS executeOperation arguments
stephenarosaj 104a1c5
npm run apidocs for validateAdminArgs
stephenarosaj 652c059
update validateAdminArgs documentation
stephenarosaj 79b55eb
address validateAdminArgs and some test comments
stephenarosaj 97f75f3
update validate-admin-args and add tests, address existing test comme…
stephenarosaj f7d2a5b
update tests
stephenarosaj e95a035
update tests
stephenarosaj 4e5fbcb
address mutation test comments
stephenarosaj b83b6da
address prod url comments
stephenarosaj 39cfb9d
finally fixed unit tests
stephenarosaj 5d4cbab
REALLY fixed unit tests
stephenarosaj File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
{ | ||
"name": "firebase-admin", | ||
"version": "13.5.0", | ||
"version": "13.4.0", | ||
"description": "Firebase admin SDK for Node.js", | ||
"author": "Firebase <[email protected]> (https://firebase.google.com/)", | ||
"license": "Apache-2.0", | ||
|
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lahirumaramba Any opinions on this? We only need this for the generated SDK, and we don't intend for developers to use this.