-
Notifications
You must be signed in to change notification settings - Fork 199
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
Template ID to JsString JSON serialization #3991
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
leo-da
changed the title
Template ID to JsString in JSON serialization
Template ID to JsString JSON serialization
Jan 8, 2020
hurryabit
force-pushed
the
leo-3647-template-id-str
branch
from
January 9, 2020 16:53
d641f9f
to
cba72ed
Compare
…eo-3647-template-id-str
CHANGELOG_BEGIN [JSON API - Experimental] - Use JSON string to encode template IDs. Use colon (``:``) to separate parts of the ID. The request format, with optional package ID: - "<module>:<entity>" - "<package ID>:<module>:<entity>" The response always contains fully qualified template ID in the format: - "<package ID>:<module>:<entity>" CHANGELOG_END
ghost
reviewed
Jan 9, 2020
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.
daml2ts changes lgtm
ghost
approved these changes
Jan 9, 2020
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.
I see @hurryabit has provided implicit approval for the change generally. Accordingly, I'm marking this approved.
S11001001
approved these changes
Jan 9, 2020
S11001001
added a commit
that referenced
this pull request
Jan 10, 2020
S11001001
added a commit
that referenced
this pull request
Jan 15, 2020
* minor improvements on websocket and add websocketIT * add it for websocket, and support config args * add one more test case * make ws config optional * avoid fromTryCatchNonFatal when derivative already exists * spelling and missing type parameter * use richer Matchers in WebsocketServiceIntegrationTest * scalafmt * IDEs may love braces but we don't * utility for simplifying FanOutShape2s; use in ContractsService * split matSecondOut into generalization; make compile again * match matSecondOut utility with standard utility methods * spelling * getCreatesAndArchivesSince doesn't need to query the transaction boundary * boolean newtype utility * split up transactionMessageHandler into components * decodeAndParsePayload passes through the Jwt * clean up config and default WS config * take multiple template IDs for insertDeleteStepSource * replace websocket return with {errors, add, remove}, based on acsFollowingAndBoundary * parse ValuePredicate in websocket * remove unused lfvToJson * nominal internal state for emitted WS steps-and-errors * missing copyright headers * add filtering to convertFilterContracts * add step conflation to websocket output * move conflation to static function * rename /transactions endpoint to /contracts/searchForever * empty requests are not allowed; numConns is per-service * option for GetCreatesAndArchiveSince to not terminate; use in WebsocketService * start of searchForever documentation * stub searchForever longer test * use valueOr * don't run all other tests again with WebsocketServiceIntegrationTest * start of websocket delta test * solve init order problem with AbstractHttpServiceIntegrationTestFuns - previous order caused test set to be cleared; mutation is intuitive for sure! * full flow test, fails for lack of create/exercise yet * passing full flow test * full documentation examples * rename add/remove to created/archived * cleaner NewBoolean.Named * document heartbeats * document subprotocols for searchForever * note about the tests mysteriously terminating * ensure create has happened before attempting query in tests * reorganize multi-step WS test so its states and assertions are clearer * filter out heartbeats in raw string tests * factor out ContractDelta * make exercisePayload easier to read * filter out heartbeats in conversation test * remove type lambda * accept chunked queries - clients may not be in control of how query bodies are delivered to the server, so we should be agnostic in that respect * add changelog CHANGELOG_BEGIN - [JSON API - Experimental] WebSocket contract search at ``/contracts/searchForever``. See `issue #3936 <https://github.com/digital-asset/daml/pull/3936>`_. CHANGELOG_END * adapt to #3991 template ID strings * adapt to #3971 argument -> payload * fix create command for test (string template ID redux) * adapt to #4014 ResolveTemplateId change * update copyright headers * rebuild WS example output to match latest changes - thanks @leo-da * SeqOps is not a safe name * don't need breakOut anymore * use util library form of partitionMap - thanks @leo-da for pointing it out Co-authored-by: lima-da <54044170+lima-da@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes: #3647
TODO:
Pull Request Checklist
CHANGELOG_BEGIN
andCHANGELOG_END
tagsNOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with
/AzurePipelines run
totrigger the build.