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

WIP: feature/change-consent-detail-to-read-only #7277

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chopkinsmade
Copy link
Contributor

@chopkinsmade chopkinsmade commented Oct 31, 2024

Description of change

Add a new table on the contact page showing their consent data

Test instructions

Running the frontend locally, connected to a local instance of the API, these 2 examples can be used for testing:

Screenshots

Before

image

After

Add a screenshot

Checklist

  • Has the branch been rebased to main?
  • Automated tests (Any of the following when applicable: Unit, Functional or End-to-End)
  • Manual compatibility testing (Browsers: Chrome, Firefox, Edge, Safari)

@chopkinsmade chopkinsmade changed the title feature/change-consent-detail-to-read-only WIP: feature/change-consent-detail-to-read-only Oct 31, 2024
Copy link

cypress bot commented Oct 31, 2024

data-hub-frontend    Run #57895

Run Properties:  status check passed Passed #57895  •  git commit af61a75374: Add contact consent to sandbox
Project data-hub-frontend
Branch Review feature/change-contact-consent-to-read-only
Run status status check passed Passed #57895
Run duration 01m 39s
Commit git commit af61a75374: Add contact consent to sandbox
Committer Chris Hopkins
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 15
View all changes introduced in this branch ↗︎

@chopkinsmade chopkinsmade force-pushed the feature/change-consent-detail-to-read-only branch 9 times, most recently from 8f85ff3 to 3cce0a0 Compare November 5, 2024 10:05
@chopkinsmade chopkinsmade force-pushed the feature/change-consent-detail-to-read-only branch 2 times, most recently from 0a406de to 80650a2 Compare November 7, 2024 15:42
@chopkinsmade chopkinsmade force-pushed the feature/change-consent-detail-to-read-only branch from 80650a2 to af61a75 Compare December 11, 2024 14:07
Copy link

codecov bot commented Dec 11, 2024

Codecov Report

Attention: Patch coverage is 90.69767% with 4 lines in your changes missing coverage. Please review.

Project coverage is 88.00%. Comparing base (2126dfa) to head (af61a75).

Files with missing lines Patch % Lines
...modules/Contacts/ContactDetails/ConsentDetails.jsx 87.87% 4 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7277   +/-   ##
=======================================
  Coverage   87.99%   88.00%           
=======================================
  Files        1168     1170    +2     
  Lines       18171    18214   +43     
  Branches     5139     5153   +14     
=======================================
+ Hits        15990    16029   +39     
- Misses       2181     2185    +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant