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

Removed the terms and condition text on login page #1487

Merged
merged 7 commits into from
May 7, 2024

Conversation

Yagnik56
Copy link
Collaborator

@Yagnik56 Yagnik56 commented May 1, 2024

In this PR terms and condition text on login page below Google sign in button has been removed as it looked like clickable link but didn't redirect any page and we don't have any term & conditions of our own as we are using google sign in only.

@Yagnik56 Yagnik56 requested a review from zackcl May 1, 2024 08:16
@Yagnik56 Yagnik56 self-assigned this May 1, 2024
@danoswaltCL
Copy link
Collaborator

seems like the right thing to do to me, @amurphy-cl thoughts on just removing the link?

@zackcl
Copy link
Collaborator

zackcl commented May 1, 2024

Yeah, I also think it's fine to remove it. Or maybe replace the text with something else like "To learn more about UpGrade, please visit www.upgradeplatform.org".

Before:

Screenshot 2024-05-01 at 3 24 57 PM

After:

Screenshot 2024-05-01 at 3 24 15 PM

Demo App Login (For reference):

Screenshot 2024-05-01 at 3 27 18 PM

@amurphy-cl
Copy link
Collaborator

Linking to the website seems fine.

@zackcl zackcl requested a review from danoswaltCL May 1, 2024 19:58
@zackcl
Copy link
Collaborator

zackcl commented May 1, 2024

@amurphy-cl I've added the new info message with a link to www.upgradeplatform.org. @danoswaltCL Feel free to review the change I made.

Screenshot 2024-05-01 at 3 58 40 PM

@zackcl zackcl merged commit abd6227 into dev May 7, 2024
8 checks passed
@zackcl zackcl deleted the bugfix/remove-term-and-condition-text-on-login-page branch May 7, 2024 19:38
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.

Terms and Conditions Link is not clickable and Content for the same is not present
5 participants