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

Eula popup added #1842

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Eula popup added #1842

merged 3 commits into from
Sep 23, 2024

Conversation

AadeshNichite
Copy link
Collaborator

@AadeshNichite AadeshNichite commented Sep 10, 2024

Eula popup added while sign in and sign up

Related Tasks:
https://chefio.atlassian.net/browse/CHEF-13951

Screen.Recording.2024-09-10.at.5.13.18.PM.mov

Signed-off-by: AadeshNichite <[email protected]>
Signed-off-by: AadeshNichite <[email protected]>
@sajjaphani
Copy link
Contributor

@AadeshNichite If a user accepts the EULA during either sign-in or sign-up, do we need to show it again for the other option? Can you check with Product?

@AadeshNichite
Copy link
Collaborator Author

@sajjaphani I had a discussion with product regarding when to show Popup. We have to show popup at the time of sign in only.

@sajjaphani sajjaphani merged commit 99b66a9 into main Sep 23, 2024
6 of 7 checks passed
@sajjaphani sajjaphani deleted the Aadesh/Chef-13951 branch September 23, 2024 10:33
chef-expeditor bot pushed a commit that referenced this pull request Sep 23, 2024
Obvious fix; these changes are the result of automation not creative thinking.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants