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

Add Terms of Service #4065

Merged
merged 3 commits into from
May 31, 2023
Merged

Add Terms of Service #4065

merged 3 commits into from
May 31, 2023

Conversation

kflemin
Copy link
Contributor

@kflemin kflemin commented May 30, 2023

What's this PR do?

Add Legal Terms to landing page

How should this be manually tested?

go to the login page or the "home" page when not logged in. you should see:

Screen Shot 2023-05-30 at 2 45 07 PM

Then click on "View/Hide Terms" to reveal:
Screen Shot 2023-05-30 at 2 45 18 PM

What are the relevant tickets?

#3964

@kflemin kflemin added the Feature Add this label to new features. This will be reflected in the change log when generated. label May 30, 2023
@axelstudios axelstudios changed the title adding terms section to landing page Add Terms of Service May 31, 2023
@axelstudios
Copy link
Member

axelstudios commented May 31, 2023

@kflemin I added some style improvements, and got rid of the duplicative login.html page.

We probably need to update the terms that are linked from the About page also

image

@axelstudios axelstudios merged commit e8b1fd4 into develop May 31, 2023
7 checks passed
@axelstudios axelstudios deleted the terms-and-conditions branch May 31, 2023 20:21
dhaley pushed a commit that referenced this pull request Jul 21, 2023
* adding terms section to landing page

* Cleanup, consolidation

* Update Terms_of_Service.txt

---------

Co-authored-by: Alex Swindler <Alex.Swindler@nrel.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Add this label to new features. This will be reflected in the change log when generated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants