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

Fix licensing information to align with legal terms #338

Merged
merged 2 commits into from
Jun 18, 2024

Conversation

keirthana
Copy link
Contributor

Done

For documenration, updated license information to CC-BY-SA according to terms confirmed with legal team.

QA

  • Check out this feature branch
  • Run the site using the command ./run serve
  • View the site locally in your web browser at: http://0.0.0.0:8043/
  • Run through the following QA steps
  • [List additional steps to QA the new features or prove the bug has been resolved]

Issue / Card

Fixes #

Screenshots

[if relevant, include a screenshot]

@webteam-app
Copy link

Copy link
Contributor

@edlerd edlerd left a comment

Choose a reason for hiding this comment

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

The copyright now spans two lines, breaking within the license name:

image

Maybe we can rearrange the text a bit like this:

image

Not sure how flexible the text is legally. Or add a line break after the year, that improves as well:

image

@keirthana
Copy link
Contributor Author

The copyright now spans two lines, breaking within the license name:

image

Maybe we can rearrange the text a bit like this:

image

Not sure how flexible the text is legally. Or add a line break after the year, that improves as well:

image

I was trying to see if I can make the footer content two column instead of three. However, this is better. I have chosen the latter option so that we keep the text intact.

@keirthana keirthana force-pushed the fix-licensing-message branch from 11c2daf to 905933f Compare June 18, 2024 06:05
@keirthana keirthana requested a review from edlerd June 18, 2024 06:14
@edlerd edlerd merged commit 4e2be5b into canonical:main Jun 18, 2024
6 checks passed
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.

3 participants