-
Notifications
You must be signed in to change notification settings - Fork 105
Use REST interface for reporting calls #507
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
avermeil
wants to merge
38
commits into
master
Choose a base branch
from
rest-reports
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.
Open
Changes from 1 commit
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
eaa89ee
loose attempt at using the much more performant REST interface for re…
avermeil abe39ff
convert more functions to REST
avermeil ea18c81
fix for summary row in multi page report calls
avermeil 17179ea
add support for max_reporting_rows
avermeil dbcd84a
attempt to parse deep messages (not yet working)
avermeil abf3d59
Merge branch 'master' into rest-reports
avermeil dc857a5
use protobuf's lookup fn to build path-to-fieldType object for parsing
avermeil dea642a
adjust tests
avermeil 04620df
minor cleanup
avermeil 639d1b0
fix for enums not in mega
avermeil 42479c0
minor cleanup
avermeil 4fa4623
remove logging
avermeil ab68954
better handling of unknown errors
avermeil 18bba8c
remove redundant auth in REST headers
avermeil e8a3074
fix issue with totalResultsCount
avermeil d9ca0ba
fix tests
avermeil 87007da
Merge branch 'master' into rest-reports
avermeil e68425f
run yarn compile
avermeil a1cb776
Merge branch 'v19' into rest-reports
avermeil dce2c55
remove TODO
avermeil e1a48ae
minor adjustments to reflect new search_settings
avermeil 1b0407e
Test fixes
FidelElie ebe68f0
Merge branch 'master' into rest-reports
FidelElie 90db605
Merge branch 'master' into rest-reports
FidelElie 2fea840
Fix error for omitted return_total_results field
FidelElie 9e4cc97
Add back return_total_results_count request option
FidelElie f61b500
fix search_settings example
Lewis-Clayton cd69e5a
fix for .reportCount()
avermeil 1f28ffe
escape query constraints
Lewis-Clayton e86164b
better handling of undefined results from stream
avermeil 8daa3c7
bump version
avermeil 8940c99
Comply with tests
Lewis-Clayton 25247eb
Merge pull request #511 from Opteo/escaping-query-constraints
Lewis-Clayton 6721282
Bump Version
Lewis-Clayton 9be5bc0
v20 update
Lewis-Clayton 97922b9
prettified
Lewis-Clayton 9031b45
merge v20 in to rest-reports
Lewis-Clayton d470975
re-yarn
Lewis-Clayton 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
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.
Uh oh!
There was an error while loading. Please reload this page.