DIGI-20955 Ongoing Postbox support #199
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
…DIGI-20955_cyclic_postbox # Conflicts: # DigiMeSDK/Core/Classes/DMEPullClient.m # docs/SDKHeaders.h
…DIGI-20955_cyclic_postbox # Conflicts: # DigiMeSDK/Core/Classes/Network/DMERequestFactory.m
michaeleustace
changed the title
hamiltonalex
requested changes
Feb 15, 2021
michaeleustace
changed the base branch from
master
to
epic/DIGI-17598_cyclic_postbox
I have changed this PR to merge into epic branch. Will create separate PR for documentation in due course. |
hamiltonalex
pushed a commit
that referenced
this pull request
Mar 5, 2021
* DIGI-20955 Ongoing Postbox support (#199) * Fix some flow errors * WIP * Split common ongoing OAuth flow into separate class: DMEOAuthService * Move some more functions to OAuthService * Ensure that all pull and push client callbacks occur on main thread. * OAuthToken and OngoingPostbox now conform to NSSecureCoding. * Split action for opening digi.me app into host and path where necessary. * Bump cyclic endpoints to v1.4 * Add workspace checks * Update new SDK functions to complete on main thread * Return an SDK error if try to use ongoing postbox with one-off contract * Update Swift demo to include ongoing postbox example * Update tests * Oops. Wasn't actually pushing data using oauth token!! * DIGI-21401 Update examples app to include contacts and private keys (#201) * Add example contracts for private sharing * Add example contracts and p12s * Fix error when decoding ongoing postbox * Store ongoing postboxes in example. * Add instruction for user to try relaunching app after first ongoing postbox push * Update ongoing private sharing contract. Added info about each contract in the code. * Fix broken CA pull examples. * DIGI-21393 Ongoing Postbox documentation (#200) * Add Ongoing Postbox documentation * Add migration tips for breaking code * Update Postbox docs with breaking changes and reference ongoing gpostbox * Fix scope documentation to refer to DMESessionOptions * Fix spelling * Fix broken link in getting started * Remove unnecessary docsets * Ignore docsets in future and any intermediary files for documentation generation * Split sources & scripts for documentation generation into seperate folder * Add documentation for Swift files * Update readmes
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.
postbox/create) were not being formed correctly (e.g.://postbox%2Fcreateinstead of://postbox/createmeaning that digi.me app wasn't routing the request correctly.