This repository has been archived by the owner on Nov 30, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 16
#1008 User identity verification flow #1231
Merged
Merged
Changes from 17 commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
732c406
Update lint commands and eslint ignore
TheAndrewJackson fd533fb
Run lints and refactor privacy cards
TheAndrewJackson b5cd7ee
Refactor modal
TheAndrewJackson cf177e1
Merge branch 'main' into 1008_user_verification_code
TheAndrewJackson 437ae69
Merge branch 'main' into 1008_user_verification_code
TheAndrewJackson a0b3baf
Merge branch 'main' into 1008_user_verification_code
TheAndrewJackson f4aa306
Merge branch 'main' into 1008_user_verification_code
TheAndrewJackson 72bc6fd
Finish verification flow
TheAndrewJackson 1af812e
get config from server
TheAndrewJackson 63cf61e
Update changelong
TheAndrewJackson 5b3bc19
Fix test failures
TheAndrewJackson 6c74ec4
Format file
TheAndrewJackson 0fe2e54
Mock out route
TheAndrewJackson 190223e
Format file
TheAndrewJackson 282be39
Add code resending
TheAndrewJackson 1707ce9
Merge branch 'main' into 1008_user_verification_code
TheAndrewJackson 03b2545
Merge branch 'main' into 1008_user_verification_code
TheAndrewJackson a004ffe
Update test to use hostUrl
TheAndrewJackson 2ef1c0a
Add headers util function and PrivacyRequestStatus status enum
TheAndrewJackson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
node_modules | ||
dist | ||
.eslintrc.json | ||
next.config.js | ||
jest.config.js |
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,63 @@ | ||
import React from "react"; | ||
import { Heading, Text, Stack, Box, Image, HStack } from "@fidesui/react"; | ||
|
||
type PrivacyCardProps = { | ||
title: string; | ||
policyKey: string; | ||
iconPath: string; | ||
description: string; | ||
onOpen: (policyKey: string) => void; | ||
}; | ||
|
||
const PrivacyCard: React.FC<PrivacyCardProps> = ({ | ||
title, | ||
policyKey, | ||
iconPath, | ||
description, | ||
onOpen, | ||
}) => ( | ||
<Box | ||
as="button" | ||
key={title} | ||
bg="white" | ||
py={8} | ||
px={6} | ||
borderRadius={4} | ||
boxShadow="base" | ||
maxWidth={["100%", "100%", "100%", 304]} | ||
transition="box-shadow 50ms" | ||
cursor="pointer" | ||
userSelect="none" | ||
m={2} | ||
_hover={{ | ||
boxShadow: "complimentary-2xl", | ||
}} | ||
_focus={{ | ||
outline: "none", | ||
boxShadow: "complimentary-2xl", | ||
}} | ||
onClick={() => onOpen(policyKey)} | ||
> | ||
<Stack spacing={7}> | ||
<HStack justifyContent="center"> | ||
<Image src={iconPath} alt={description} width="54px" height="54px" /> | ||
</HStack> | ||
|
||
<Stack spacing={1} textAlign="center"> | ||
<Heading | ||
fontSize="large" | ||
fontWeight="semibold" | ||
lineHeight="28px" | ||
color="gray.600" | ||
> | ||
{title} | ||
</Heading> | ||
<Text fontSize="xs" color="gray.600"> | ||
{description} | ||
</Text> | ||
</Stack> | ||
</Stack> | ||
</Box> | ||
); | ||
|
||
export default PrivacyCard; |
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use our
${hostUrl}
const in tests too? Or is there a specific reason not to?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can. There's no particular reason. I'll update it now.