Closed Emily-ejag closed 3 months ago
Visit the preview URL for this PR (updated for commit c381dfe):
https://roar-staging--pr704-enh-adobe-sign-totrhgh5.web.app
(expires Tue, 23 Jul 2024 17:42:59 GMT)
🔥 via Firebase Hosting GitHub Action 🌎
Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460
0 | 26 | 0 | 0 | 0 |
Details:
Tests for PR 704 "WIP: AdobeSign integration" from commit "c381dfe43306ece7e367d... | |||
Project: roar-dashboard-e2e | Commit: c381dfe433 |
||
Status: Passed | Duration: 03:45 💡 | ||
Started: Jul 16, 2024 5:42 PM | Ended: Jul 16, 2024 5:46 PM |
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
@Emily-ejag , we've moved the integration token to Google Secret Manager. But there are still the old integration tokens in the git commit history. Can you look through the tokens identified here and delete them from your adobe sign developer account. Then we can dismiss the GitGuardian checks that are failing.
Created a new PR (https://github.com/yeatmanlab/roar-dashboard/pull/716) to not have security issues
Proposed changes
Please first link this branch with this PR on firekit: AdobeSign Firekit
-- Adding identical files from AdobeSign to the Legal section. -- Introducing an Adobe Sign checkbox in the Consent Picker, saving the legal documents under the legal directory in the administration. -- On the homeParticipant check, verify if the adobeSign flag exists. If it does, it ask to input your email, then you have 20 retrys of 5 sec each to sign, if not you need to start the process again. -- If the signing is successful then the student document gets updated with the consent form
ON PARENT SIGNUP for using adobe on parent signup the route is roar.education/register-clinic
Types of changes
What types of changes does this pull request introduce?
Checklist
Justification of missing checklist items
Further comments