Skip to content

MS-621 Activity contract wrapper for the intent calls#97

Merged
luhmirin-s merged 10 commits into
mainfrom
feature/MS-621-contracts
Aug 22, 2024
Merged

MS-621 Activity contract wrapper for the intent calls#97
luhmirin-s merged 10 commits into
mainfrom
feature/MS-621-contracts

Conversation

@luhmirin-s
Copy link
Copy Markdown
Contributor

@luhmirin-s luhmirin-s commented Aug 19, 2024

Major benefits of activity contact:

  • The caller app does not need to work around the activity lifecycle and "onActivityResult()" callback.
  • Type-safe interface on the call side (no need to parse intent and bundle in the calling app).
  • Also, we can pass and handle extra data within the intent without any change from the caller.

Ideally, we want all the integration partners to switch to the new API; therefore, we need to try to cover all possible use/edge cases before making it public.

@luhmirin-s luhmirin-s force-pushed the feature/MS-621-contracts branch from 1a51d4a to 5541645 Compare August 20, 2024 07:34
@luhmirin-s luhmirin-s force-pushed the feature/MS-621-contracts branch 2 times, most recently from 8c338d5 to e7099cc Compare August 20, 2024 08:47
@luhmirin-s luhmirin-s force-pushed the feature/MS-621-contracts branch from e7099cc to de3b0c3 Compare August 20, 2024 09:53
@luhmirin-s luhmirin-s requested review from a team and BurningAXE and removed request for a team August 20, 2024 10:03
@luhmirin-s luhmirin-s marked this pull request as ready for review August 20, 2024 10:03
This would allow callers to use intents directly if contracts are not possible.
@luhmirin-s
Copy link
Copy Markdown
Contributor Author

@alex-vt @alexandr-simprints I have added some important commits that are worth resetting the approvals :)

@luhmirin-s luhmirin-s merged commit 78002fc into main Aug 22, 2024
@luhmirin-s luhmirin-s deleted the feature/MS-621-contracts branch August 22, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants