Skip to content
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

Trigger signing modals from approval requests #6354

Merged

Conversation

matthewwalsh0
Copy link
Member

@matthewwalsh0 matthewwalsh0 commented May 9, 2023

Description

Trigger the signature request modals using the generic ApprovalController requests rather than events specific to the SignatureController.

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2023

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Base automatically changed from chore/785-signature-controller to main May 12, 2023 09:14
…6-trigger-signing-modals-with-approval-requests
@matthewwalsh0 matthewwalsh0 force-pushed the chore/786-trigger-signing-modals-with-approval-requests branch from 6ed59ac to bfc0a91 Compare May 12, 2023 10:23
@matthewwalsh0 matthewwalsh0 marked this pull request as ready for review May 12, 2023 10:31
@matthewwalsh0 matthewwalsh0 requested a review from a team as a code owner May 12, 2023 10:31
@matthewwalsh0 matthewwalsh0 marked this pull request as draft May 19, 2023 11:04
@matthewwalsh0 matthewwalsh0 marked this pull request as ready for review May 23, 2023 11:07
bergarces
bergarces previously approved these changes May 24, 2023
ios/Podfile.lock Outdated Show resolved Hide resolved
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@matthewwalsh0 matthewwalsh0 merged commit fc11831 into main May 24, 2023
@matthewwalsh0 matthewwalsh0 deleted the chore/786-trigger-signing-modals-with-approval-requests branch May 24, 2023 11:13
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2023
@matthewwalsh0 matthewwalsh0 added the release-6.7.0 Issue or pull request that will be included in release 6.7.0 label May 24, 2023
@cortisiko cortisiko added release-7.1.0 Issue or pull request that will be included in release 7.1.0 and removed release-6.7.0 Issue or pull request that will be included in release 6.7.0 labels May 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.1.0 Issue or pull request that will be included in release 7.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants