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

Show warning for Alpha and Beta connectors #10319

Merged
merged 8 commits into from
Feb 18, 2022

Conversation

juliachvyrova
Copy link
Contributor

@juliachvyrova juliachvyrova commented Feb 14, 2022

What

Add warning message in Source/Destination form for alpha and beta connectors
Снимок экрана 2022-02-17 в 13 32 22

Снимок экрана 2022-02-17 в 13 32 13

Closes #9712

@juliachvyrova juliachvyrova self-assigned this Feb 14, 2022
@github-actions github-actions bot added area/frontend area/platform issues related to the platform labels Feb 14, 2022
@jrhizor
Copy link
Contributor

jrhizor commented Feb 14, 2022

Sorry, I should have called it out in the issue, but from the comment https://www.figma.com/file/etaOkOsnXMoKwbfhCD71kw?node-id=30010:138107#144935197 we're looking at only having the inline warning, not the modal.

@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 10:12 Inactive
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 10:12 Inactive
@juliachvyrova juliachvyrova changed the title Acknowledgement of terms modal Acknowledgement of terms Feb 17, 2022
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 10:26 Inactive
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 10:26 Inactive
@juliachvyrova juliachvyrova force-pushed the julia/acknowledgement-of-terms-modal branch from b8dd6dc to 2d4ea01 Compare February 17, 2022 10:33
@juliachvyrova juliachvyrova force-pushed the julia/acknowledgement-of-terms-modal branch from 2d4ea01 to f3e9f2b Compare February 17, 2022 10:34
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 10:35 Inactive
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 10:35 Inactive
@juliachvyrova juliachvyrova marked this pull request as ready for review February 17, 2022 10:36
@@ -35,7 +35,7 @@ const AdditionMessage = styled.span`
`;

const LabeledToggle: React.FC<IProps> = (props) => (
<ToggleContainer>
<ToggleContainer className={props.className}>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❓ Is this a left over, or something that is fixing something?

@timroes timroes changed the title Acknowledgement of terms Show warning for Alpha and Beta connectors Feb 17, 2022
Copy link
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor code quality requests, functionally everything looks fine.

@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 19:18 Inactive
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 19:18 Inactive
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 22:05 Inactive
@juliachvyrova juliachvyrova temporarily deployed to more-secrets February 17, 2022 22:05 Inactive
@jamakase jamakase merged commit c4b172a into master Feb 18, 2022
@jamakase jamakase deleted the julia/acknowledgement-of-terms-modal branch February 18, 2022 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

require acknowledgement of terms when using alpha or beta connectors
4 participants